add unit test for streaming aggregator#682

Closed

juntao-lei-timeplus wants to merge 7 commits intodevelopfrom

test/663-unit-test-streaming-aggregator

juntao-lei-timeplus commented Apr 26, 2024

…unt,append only mode.

PR checklist:

Did you run ClangFormat ?
Did you separate headers to a different section in existing community code base ?
Did you surround proton: starts/ends for new code in existing community code base ?

Please write user-readable short description of the changes:


          add unit test for streaming aggregator, finished global and window co…

729f09d

…unt,append only mode.

juntao-lei-timeplus linked an issue

that may be closed by this pull request

Add unit test for streaming aggregator #663

Open

CLAassistant commented Apr 26, 2024 •

edited

Loading

All committers have signed the CLA.

yl-lisen reviewed

View reviewed changes

src/Interpreters/Streaming/tests/gtest_streaming_aggregator.cpp Outdated

yl-lisen reviewed

View reviewed changes

src/Interpreters/Streaming/tests/gtest_streaming_aggregator.cpp Outdated

yl-lisen reviewed

View reviewed changes

src/Interpreters/Streaming/tests/gtest_streaming_aggregator.cpp Outdated

yl-lisen reviewed

View reviewed changes

src/Interpreters/Streaming/tests/gtest_streaming_aggregator.cpp Outdated

yl-lisen reviewed

View reviewed changes

src/Interpreters/Streaming/tests/gtest_streaming_aggregator.cpp Outdated

yl-lisen reviewed

View reviewed changes

src/Interpreters/Streaming/tests/gtest_streaming_aggregator.cpp Outdated

yokofly added the pr-test label

juntao-lei-timeplus added 6 commits

April 28, 2024 16:09


          Refactoring code

54f1eba


          cover changelog input,changelog,on uedate output cases.

df714d9

fix

820c3ec


          Refactor code for improved readability and maintainability

7605b35

fix

34f4123

fix

a4e4c2d

chenziliang closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels