Skip to content

[To dev/1.3] Fix RatisConsensus client==null exception when stopping datanode #14583

Merged
HTHou merged 1 commit intodev/1.3from
fix_ratisclient_client_null_issue_cp
Dec 30, 2024
Merged

[To dev/1.3] Fix RatisConsensus client==null exception when stopping datanode #14583
HTHou merged 1 commit intodev/1.3from
fix_ratisclient_client_null_issue_cp

Conversation

@OneSizeFitsQuorum
Copy link
Contributor

No description provided.

Signed-off-by: OneSizeFitQuorum <tanxinyu@apache.org>
@sonarqubecloud
Copy link

@codecov
Copy link

codecov bot commented Dec 30, 2024

Codecov Report

Attention: Patch coverage is 50.00000% with 4 lines in your changes missing coverage. Please review.

Project coverage is 41.52%. Comparing base (1bada4a) to head (24d65b1).
Report is 1 commits behind head on dev/1.3.

Files with missing lines Patch % Lines
...g/apache/iotdb/consensus/ratis/RatisConsensus.java 50.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             dev/1.3   #14583   +/-   ##
==========================================
  Coverage      41.51%   41.52%           
  Complexity       193      193           
==========================================
  Files           3510     3510           
  Lines         225103   225106    +3     
  Branches       26795    26795           
==========================================
+ Hits           93462    93480   +18     
+ Misses        131641   131626   -15     

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

@HTHou HTHou merged commit fc9b77a into dev/1.3 Dec 30, 2024
@HTHou HTHou deleted the fix_ratisclient_client_null_issue_cp branch December 30, 2024 08:20
Caideyipi pushed a commit to Caideyipi/iotdb that referenced this pull request Mar 25, 2026
…datanode apache#14583

Signed-off-by: OneSizeFitQuorum <tanxinyu@apache.org>
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.

2 participants