-
Notifications
You must be signed in to change notification settings - Fork 11.7k
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
add import/export command and test #6972
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@carlvine500 Please check code sytle and fix it
ok |
Codecov Report
@@ Coverage Diff @@
## develop #6972 +/- ##
=============================================
+ Coverage 42.65% 42.75% +0.10%
- Complexity 9211 9260 +49
=============================================
Files 1131 1133 +2
Lines 80470 80729 +259
Branches 10517 10553 +36
=============================================
+ Hits 34322 34519 +197
- Misses 41856 41880 +24
- Partials 4292 4330 +38
... and 40 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@mxsm bazel-compile still failed , can I ignore it ? |
Yes, you can ignore it |
tools/src/main/java/org/apache/rocketmq/tools/command/message/ExportMessageCommand.java
Outdated
Show resolved
Hide resolved
tools/src/main/java/org/apache/rocketmq/tools/command/message/ExportMessageCommand.java
Show resolved
Hide resolved
@ferrirW failure test is not about export/import , please review code again . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's fixed in latest version
tools/src/main/java/org/apache/rocketmq/tools/command/message/ExportMessageCommand.java
Outdated
Show resolved
Hide resolved
tools/src/main/java/org/apache/rocketmq/tools/command/message/ExportMessageCommand.java
Show resolved
Hide resolved
tools/src/main/java/org/apache/rocketmq/tools/command/message/ExportMessageCommand.java
Outdated
Show resolved
Hide resolved
tools/src/main/java/org/apache/rocketmq/tools/command/message/ExportMessageCommand.java
Outdated
Show resolved
Hide resolved
bugfix2: offsetWriter may be closed before export all message;
This PR is stale because it has been open for 365 days with no activity. It will be closed in 3 days if no further activity occurs. If you wish not to mark it as stale, please leave a comment in this PR. |
This PR was closed because it has been inactive for 3 days since being marked as stale. |
Which Issue(s) This PR Fixes
Fixes #6971
Brief Description
grep keyword messageFile
How Did You Test This Change?
example usage