Skip to content

Pipe Consensus: Fix failed to read tsfile when counting points on receivers#12839

Merged
SteveYurongSu merged 1 commit intomasterfrom
pipe-consensus-point-counter
Jul 2, 2024
Merged

Pipe Consensus: Fix failed to read tsfile when counting points on receivers#12839
SteveYurongSu merged 1 commit intomasterfrom
pipe-consensus-point-counter

Conversation

@SteveYurongSu
Copy link
Copy Markdown
Member

As title.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Jul 2, 2024

Copy link
Copy Markdown
Collaborator

@Pengzna Pengzna left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov
Copy link
Copy Markdown

codecov bot commented Jul 2, 2024

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 41.73%. Comparing base (5a374ab) to head (9661e10).

Files Patch % Lines
.../protocol/pipeconsensus/PipeConsensusReceiver.java 0.00% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #12839      +/-   ##
============================================
- Coverage     41.73%   41.73%   -0.01%     
  Complexity       71       71              
============================================
  Files          3319     3319              
  Lines        209862   209862              
  Branches      24949    24949              
============================================
- Hits          87583    87580       -3     
- Misses       122279   122282       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@SteveYurongSu SteveYurongSu merged commit fb140af into master Jul 2, 2024
@SteveYurongSu SteveYurongSu deleted the pipe-consensus-point-counter branch July 2, 2024 13:25
JackieTien97 pushed a commit that referenced this pull request Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants