Skip to content

Conversation

@TennyZhuang
Copy link
Contributor

Signed-off-by: TennyZhuang [email protected]

What's changed and what's your intention?

Checklist

  • I have written necessary docs and comments
  • I have added necessary unit tests and integration tests

Refer to a related PR or issue link (optional)

Fix #54

@github-actions github-actions bot added the type/feature Type: New feature. label Jan 29, 2022
Signed-off-by: TennyZhuang <[email protected]>
Signed-off-by: TennyZhuang <[email protected]>
Signed-off-by: TennyZhuang <[email protected]>
Signed-off-by: TennyZhuang <[email protected]>
@codecov
Copy link

codecov bot commented Jan 29, 2022

Codecov Report

Merging #58 (1531ee0) into main (9553f97) will decrease coverage by 0.00%.
The diff coverage is 93.33%.

Impacted file tree graph

@@             Coverage Diff              @@
##               main      #58      +/-   ##
============================================
- Coverage     74.48%   74.47%   -0.01%     
  Complexity     2645     2645              
============================================
  Files           825      825              
  Lines         47323    47329       +6     
  Branches       1560     1560              
============================================
  Hits          35250    35250              
- Misses        11275    11281       +6     
  Partials        798      798              
Flag Coverage Δ
java 61.91% <ø> (ø)
rust 79.98% <93.33%> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
rust/meta/src/stream/graph/stream_graph.rs 89.83% <ø> (-3.62%) ⬇️
rust/meta/src/stream/test_fragmenter.rs 98.46% <ø> (ø)
rust/meta/src/stream/fragmenter.rs 90.36% <92.68%> (-2.89%) ⬇️
rust/meta/src/stream/graph/fragment_graph.rs 100.00% <100.00%> (ø)
rust/common/src/types/ordered_float.rs 25.00% <0.00%> (-0.34%) ⬇️
rust/stream/src/executor/barrier_align.rs 78.78% <0.00%> (ø)

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 9553f97...1531ee0. Read the comment docs.

Copy link
Member

@yezizp2012 yezizp2012 left a comment

Choose a reason for hiding this comment

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

LGTM

@wyhyhyhyh
Copy link
Contributor

LGTM

@TennyZhuang TennyZhuang merged commit d24c47d into main Jan 30, 2022
@TennyZhuang TennyZhuang deleted the feat/parallel-materialize branch January 30, 2022 02:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type/feature Type: New feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support parallel MaterializeExecutor and read from it

4 participants