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

Put the LMQ prefix when storing the Offset Topic of LMQ #151

Merged
merged 2 commits into from
Oct 6, 2022

Conversation

DongyuanPan
Copy link
Contributor

@codecov-commenter
Copy link

Codecov Report

Merging #151 (5be9909) into develop (9b1f8d1) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop     #151   +/-   ##
========================================
  Coverage    41.97%   41.97%           
========================================
  Files          118      118           
  Lines         4898     4898           
  Branches       713      713           
========================================
  Hits          2056     2056           
  Misses        2488     2488           
  Partials       354      354           
Impacted Files Coverage Δ
.../rocketmq/mqtt/ds/store/LmqOffsetStoreManager.java 63.01% <100.00%> (ø)

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

@pingww
Copy link
Contributor

pingww commented Sep 29, 2022

add some tests

@pingww pingww merged commit 41b6e8b into apache:develop Oct 6, 2022
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.

4 participants