Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add runner for RequestAggUnion and Reduce #1532

Merged
merged 7 commits into from
Apr 2, 2022

Conversation

zhanghaohit
Copy link
Collaborator

@zhanghaohit zhanghaohit commented Mar 29, 2022

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    runners for long-window

close #1209
close #1210
close #1549
close #1564

@zhanghaohit zhanghaohit marked this pull request as draft March 29, 2022 16:08
@github-actions
Copy link
Contributor

github-actions bot commented Mar 29, 2022

Linux Test Report

       54 files       181 suites   41m 49s ⏱️
  8 191 tests   8 191 ✔️ 0 💤 0
12 107 runs  12 107 ✔️ 0 💤 0

Results for commit 6ecf98d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 29, 2022

SDK Test Report

  72 files    72 suites   7m 27s ⏱️
172 tests 169 ✔️ 3 💤 0
212 runs  209 ✔️ 3 💤 0

Results for commit 6ecf98d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 29, 2022

HybridSE Linux Test Report

       65 files       232 suites   5m 8s ⏱️
18 870 tests 18 870 ✔️ 0 💤 0

Results for commit 6ecf98d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 29, 2022

HybridSE Mac Test Report

       65 files       232 suites   6m 35s ⏱️
18 870 tests 18 870 ✔️ 0 💤 0

Results for commit 6ecf98d.

♻️ This comment has been updated with latest results.

@codecov
Copy link

codecov bot commented Mar 29, 2022

Codecov Report

Merging #1532 (6ecf98d) into main (af937e5) will increase coverage by 0.48%.
The diff coverage is 84.72%.

@@             Coverage Diff              @@
##               main    #1532      +/-   ##
============================================
+ Coverage     66.50%   66.98%   +0.48%     
- Complexity      281      323      +42     
============================================
  Files           583      597      +14     
  Lines        110471   113110    +2639     
  Branches        980      994      +14     
============================================
+ Hits          73465    75769    +2304     
- Misses        36814    37134     +320     
- Partials        192      207      +15     
Impacted Files Coverage Δ
...ybridse/examples/toydb/src/tablet/tablet_catalog.h 63.15% <0.00%> (-3.51%) ⬇️
hybridse/include/sdk/base.h 19.23% <ø> (ø)
hybridse/include/vm/schemas_context.h 100.00% <ø> (ø)
hybridse/include/vm/simple_catalog.h 80.00% <ø> (ø)
hybridse/src/vm/sql_compiler.h 66.66% <ø> (+5.12%) ⬆️
hybridse/src/vm/transform.h 92.00% <ø> (ø)
src/cmd/sql_cmd_test.cc 100.00% <ø> (ø)
src/tablet/tablet_impl.h 100.00% <ø> (ø)
src/catalog/base.h 70.58% <66.66%> (+6.95%) ⬆️
src/tablet/tablet_impl.cc 42.13% <66.66%> (+3.29%) ⬆️
... and 92 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 230f99b...6ecf98d. Read the comment docs.

@zhanghaohit
Copy link
Collaborator Author

@zhanghaohit zhanghaohit marked this pull request as ready for review March 30, 2022 01:11
Copy link
Collaborator

@tobegit3hub tobegit3hub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tobegit3hub tobegit3hub merged commit d530c7e into 4paradigm:main Apr 2, 2022
@lumianph lumianph mentioned this pull request May 13, 2022
39 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants