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

[ISSUE #7437] Add the CRC check of commitlog #7468

Merged
merged 6 commits into from
Oct 25, 2023

Conversation

guyinyou
Copy link
Contributor

Which Issue(s) This PR Fixes

Fixes #7437

Brief Description

How Did You Test This Change?

@@ -61,7 +61,6 @@
import org.apache.rocketmq.store.logfile.MappedFile;
import org.apache.rocketmq.store.util.LibC;
import org.rocksdb.RocksDBException;

Copy link
Contributor

Choose a reason for hiding this comment

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

maybe no need to reformat the import in this file and MessageConfig.java

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

Copy link
Contributor

@RongtongJin RongtongJin left a comment

Choose a reason for hiding this comment

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

Add some UTs

@guyinyou
Copy link
Contributor Author

Add some UTs

done

@codecov-commenter
Copy link

Codecov Report

Merging #7468 (401e636) into develop (38d2676) will decrease coverage by 0.01%.
Report is 1 commits behind head on develop.
The diff coverage is 64.17%.

@@              Coverage Diff              @@
##             develop    #7468      +/-   ##
=============================================
- Coverage      43.03%   43.02%   -0.01%     
- Complexity      9672     9681       +9     
=============================================
  Files           1158     1158              
  Lines          83858    83978     +120     
  Branches       10889    10915      +26     
=============================================
+ Hits           36090    36135      +45     
- Misses         43307    43351      +44     
- Partials        4461     4492      +31     
Files Coverage Δ
...g/apache/rocketmq/common/message/MessageConst.java 98.14% <100.00%> (+0.03%) ⬆️
...ache/rocketmq/store/config/MessageStoreConfig.java 57.51% <100.00%> (+0.49%) ⬆️
...org/apache/rocketmq/container/BrokerContainer.java 53.24% <0.00%> (-0.25%) ⬇️
...ache/rocketmq/container/BrokerContainerConfig.java 87.87% <75.00%> (-1.78%) ⬇️
...rocketmq/common/message/MessageExtBrokerInner.java 56.00% <88.46%> (+56.00%) ⬆️
...a/org/apache/rocketmq/store/MessageExtEncoder.java 85.43% <64.70%> (-4.65%) ⬇️
.../main/java/org/apache/rocketmq/common/UtilAll.java 36.23% <0.00%> (-0.61%) ⬇️
...main/java/org/apache/rocketmq/store/CommitLog.java 58.93% <80.00%> (+0.97%) ⬆️
...apache/rocketmq/common/message/MessageDecoder.java 60.05% <0.00%> (-3.51%) ⬇️

... and 24 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Contributor

@joeCarf joeCarf left a comment

Choose a reason for hiding this comment

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

lgtm

@RongtongJin RongtongJin merged commit 91349f3 into apache:develop Oct 25, 2023
9 of 10 checks passed
Copy link
Contributor Author

@guyinyou guyinyou left a comment

Choose a reason for hiding this comment

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

lgtm

@guyinyou guyinyou deleted the dev_gyy_crc branch November 8, 2023 02:39
@guyinyou guyinyou restored the dev_gyy_crc branch November 8, 2023 02:39
@guyinyou guyinyou deleted the dev_gyy_crc branch November 8, 2023 02:39
@guyinyou guyinyou restored the dev_gyy_crc branch November 8, 2023 02:39
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.

[Bug] A bug in the CRC check of commitlog
4 participants