[GitHub] [rocketmq-client-cpp] ThreeAndTwo edited a comment on issue #355: Failed to build Boost.Build build engine

2021-08-10 Thread GitBox


ThreeAndTwo edited a comment on issue #355:
URL: 
https://github.com/apache/rocketmq-client-cpp/issues/355#issuecomment-896474315


   I try to change boost version to 1.76.0, It's worked! and then I find the 
default c++ compiler is clang, I already switch c++ compiler to gcc-11 but 
doesn't seem to work.
   
![1628651748180](https://user-images.githubusercontent.com/35927054/128965786-a9654ccd-2634-4c7a-9806-6375bf6450fb.jpg)
   
![image](https://user-images.githubusercontent.com/35927054/128965882-b4fa72aa-f79b-4235-8a9b-0e1274722260.png)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] AbelSara commented on a change in pull request #3239: use the StringBuilder another method can reduce impact on array copy performance

2021-08-10 Thread GitBox


AbelSara commented on a change in pull request #3239:
URL: https://github.com/apache/rocketmq/pull/3239#discussion_r686472902



##
File path: store/src/main/java/org/apache/rocketmq/store/index/IndexFile.java
##
@@ -71,6 +72,9 @@ public void load() {
 this.indexHeader.load();
 }
 
+/**
+ * 头部文件刷盘 时间戳 msgId 物理偏移量..

Review comment:
   Why do you use so many annotations, and it is in Chinese?




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-client-cpp] ThreeAndTwo commented on issue #355: Failed to build Boost.Build build engine

2021-08-10 Thread GitBox


ThreeAndTwo commented on issue #355:
URL: 
https://github.com/apache/rocketmq-client-cpp/issues/355#issuecomment-896487979


   @WanZixin @ifplusor I found another case and it's worked! 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-client-cpp] ThreeAndTwo edited a comment on issue #355: Failed to build Boost.Build build engine

2021-08-10 Thread GitBox


ThreeAndTwo edited a comment on issue #355:
URL: 
https://github.com/apache/rocketmq-client-cpp/issues/355#issuecomment-896487979


   @WanZixin @ifplusor I found another case and it's worked! 
   ```
   mkdir cppsdk
   cd cppsdk
   wget 
https://github.com/apache/rocketmq-client-cpp/releases/download/2.0.1/rocketmq-client-cpp-2.0.1-bin-release-darwin.tar.gz
   tar -xzf rocketmq-client-cpp-2.0.1-bin-release-darwin.tar.gz
   cp rocketmq-client-cpp/lib/* /usr/local/lib/
   mkdir -p /usr/local/include/rocketmq/
   cp rocketmq-client-cpp/include/* /usr/local/include/rocketmq/
   install_name_tool -id "@rpath/librocketmq.dylib" 
/usr/local/lib/librocketmq.dylib
   ```
   
   more info: 
https://help.aliyun.com/document_detail/157155.html?spm=a2c4g.11186623.6.702.313a5b06QfVEYE
  


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] ltamber commented on pull request #3246: Use df's algorithm to calculate the disk used ratio

2021-08-10 Thread GitBox


ltamber commented on pull request #3246:
URL: https://github.com/apache/rocketmq/pull/3246#issuecomment-896490924


   nice work.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] gylang commented on issue #3243: use the StringBuilder another method can reduce impact on array copy performance

2021-08-10 Thread GitBox


gylang commented on issue #3243:
URL: https://github.com/apache/rocketmq/issues/3243#issuecomment-896493545


   
![image](https://user-images.githubusercontent.com/41338363/128970167-20cbf379-396f-4724-9e2b-354d017b6476.png)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] chengshanbin commented on pull request #3246: Use df's algorithm to calculate the disk used ratio

2021-08-10 Thread GitBox


chengshanbin commented on pull request #3246:
URL: https://github.com/apache/rocketmq/pull/3246#issuecomment-896553051


   LGTM


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] gylang commented on a change in pull request #3239: use the StringBuilder another method can reduce impact on array copy performance

2021-08-11 Thread GitBox


gylang commented on a change in pull request #3239:
URL: https://github.com/apache/rocketmq/pull/3239#discussion_r686560208



##
File path: store/src/main/java/org/apache/rocketmq/store/index/IndexFile.java
##
@@ -71,6 +72,9 @@ public void load() {
 this.indexHeader.load();
 }
 
+/**
+ * 头部文件刷盘 时间戳 msgId 物理偏移量..

Review comment:
   sory, I don't know why this code was submitted here




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] gylang commented on a change in pull request #3239: use the StringBuilder another method can reduce impact on array copy performance

2021-08-11 Thread GitBox


gylang commented on a change in pull request #3239:
URL: https://github.com/apache/rocketmq/pull/3239#discussion_r686567859



##
File path: store/src/main/java/org/apache/rocketmq/store/index/IndexFile.java
##
@@ -71,6 +72,9 @@ public void load() {
 this.indexHeader.load();
 }
 
+/**
+ * 头部文件刷盘 时间戳 msgId 物理偏移量..

Review comment:
   this code just my study comment. i do'not want to pr the "3eee57d"




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] gylang commented on a change in pull request #3239: use the StringBuilder another method can reduce impact on array copy performance

2021-08-11 Thread GitBox


gylang commented on a change in pull request #3239:
URL: https://github.com/apache/rocketmq/pull/3239#discussion_r686568525



##
File path: store/src/main/java/org/apache/rocketmq/store/index/IndexFile.java
##
@@ -71,6 +72,9 @@ public void load() {
 this.indexHeader.load();
 }
 
+/**
+ * 头部文件刷盘 时间戳 msgId 物理偏移量..

Review comment:
   this code just my study comment. i don't want to pr "3eee57d"




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] coveralls edited a comment on pull request #3149: [ISSUE #3148]Support metadata export

2021-08-11 Thread GitBox


coveralls edited a comment on pull request #3149:
URL: https://github.com/apache/rocketmq/pull/3149#issuecomment-880750487


   
   [![Coverage 
Status](https://coveralls.io/builds/42057643/badge)](https://coveralls.io/builds/42057643)
   
   Coverage increased (+0.1%) to 54.183% when pulling 
**f646fcc57f8f9100bd1ce4ed1935ddeb4b551f36 on panzhi33:develop** into 
**e4daba5ff5e4b51a81ad690b678370adb3d00632 on apache:develop**.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] codecov-commenter edited a comment on pull request #3149: [ISSUE #3148]Support metadata export

2021-08-11 Thread GitBox


codecov-commenter edited a comment on pull request #3149:
URL: https://github.com/apache/rocketmq/pull/3149#issuecomment-889850553


   # 
[Codecov](https://codecov.io/gh/apache/rocketmq/pull/3149?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 Report
   > Merging 
[#3149](https://codecov.io/gh/apache/rocketmq/pull/3149?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 (f646fcc) into 
[develop](https://codecov.io/gh/apache/rocketmq/commit/30058cffa5a2512894a61b5c5eb97a58ce6681ec?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 (30058cf) will **increase** coverage by `0.10%`.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/rocketmq/pull/3149/graphs/tree.svg?width=650&height=150&src=pr&token=4w0sxP1wZv&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)](https://codecov.io/gh/apache/rocketmq/pull/3149?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   
   ```diff
   @@  Coverage Diff  @@
   ## develop#3149  +/-   ##
   =
   + Coverage  47.82%   47.93%   +0.10% 
   + Complexity  4551 4331 -220 
   =
 Files552  514  -38 
 Lines  3664334972-1671 
 Branches4846 4661 -185 
   =
   - Hits   1752616763 -763 
   + Misses 1688116027 -854 
   + Partials2236 2182  -54 
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/rocketmq/pull/3149?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 | Coverage Δ | |
   |---|---|---|
   | 
[...rocketmq/broker/pagecache/ManyMessageTransfer.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-YnJva2VyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9yb2NrZXRtcS9icm9rZXIvcGFnZWNhY2hlL01hbnlNZXNzYWdlVHJhbnNmZXIuamF2YQ==)
 | `36.36% <0.00%> (-9.80%)` | :arrow_down: |
   | 
[.../rocketmq/broker/pagecache/OneMessageTransfer.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-YnJva2VyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9yb2NrZXRtcS9icm9rZXIvcGFnZWNhY2hlL09uZU1lc3NhZ2VUcmFuc2Zlci5qYXZh)
 | `24.00% <0.00%> (-9.34%)` | :arrow_down: |
   | 
[...ketmq/client/impl/consumer/PullMessageService.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-Y2xpZW50L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9yb2NrZXRtcS9jbGllbnQvaW1wbC9jb25zdW1lci9QdWxsTWVzc2FnZVNlcnZpY2UuamF2YQ==)
 | `71.11% <0.00%> (-4.45%)` | :arrow_down: |
   | 
[...ketmq/common/protocol/body/ConsumerConnection.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9yb2NrZXRtcS9jb21tb24vcHJvdG9jb2wvYm9keS9Db25zdW1lckNvbm5lY3Rpb24uamF2YQ==)
 | `95.83% <0.00%> (-4.17%)` | :arrow_down: |
   | 
[...rg/apache/rocketmq/remoting/netty/NettyLogger.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-cmVtb3Rpbmcvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3JvY2tldG1xL3JlbW90aW5nL25ldHR5L05ldHR5TG9nZ2VyLmphdmE=)
 | `16.32% <0.00%> (-2.73%)` | :arrow_down: |
   | 
[...mq/client/impl/consumer/RebalanceLitePullImpl.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-Y2xpZW50L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9yb2NrZXRtcS9jbGllbnQvaW1wbC9jb25zdW1lci9SZWJhbGFuY2VMaXRlUHVsbEltcGwuamF2YQ==)
 | `48.52% <0.00%> (-2.22%)` | :arrow_down: |
   | 
[...ava/org/apache/rocketmq/filter/util/BitsArray.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar

[GitHub] [rocketmq-spring] RongtongJin merged pull request #372: [ISSUE #371] Add some asynchronous API for batch messages in RocketMQ…

2021-08-11 Thread GitBox


RongtongJin merged pull request #372:
URL: https://github.com/apache/rocketmq-spring/pull/372


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] coveralls commented on pull request #1710: [ISSUE #1709] Call doAfterRpcHooks when request is asynchronous

2021-08-11 Thread GitBox


coveralls commented on pull request #1710:
URL: https://github.com/apache/rocketmq/pull/1710#issuecomment-896674316


   
   [![Coverage 
Status](https://coveralls.io/builds/42051586/badge)](https://coveralls.io/builds/42051586)
   
   Coverage increased (+0.1%) to 54.022% when pulling 
**e4298bb5bbd1f64c9dee771686f589351b6f6c14 on aaron-ai:pr** into 
**a20f31bb3de242756542c552d7212a7a4ba4 on apache:master**.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] pirateskipper opened a new pull request #3247: Safely delete parameters and variables that have never been used

2021-08-11 Thread GitBox


pirateskipper opened a new pull request #3247:
URL: https://github.com/apache/rocketmq/pull/3247


   **Make sure set the target branch to `develop`**
   
   ## What is the purpose of the change
   
   - Streamline code and increase readability
   
   ## Brief changelog
   
   - Safely delete parameters and variables that have never been used
   - Related change files:DefaultRequestProcessor && KVConfigManager
   
   ## Verifying this change
   
   - Verified
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-client-cpp] ThreeAndTwo commented on issue #355: Failed to build Boost.Build build engine

2021-08-11 Thread GitBox


ThreeAndTwo commented on issue #355:
URL: 
https://github.com/apache/rocketmq-client-cpp/issues/355#issuecomment-896687235


   Now it does not work! How should to resolve it???


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] pirateskipper closed pull request #3247: Safely delete parameters and variables that have never been used

2021-08-11 Thread GitBox


pirateskipper closed pull request #3247:
URL: https://github.com/apache/rocketmq/pull/3247


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] pirateskipper opened a new pull request #3248: Safely delete parameters and variables that have never been used

2021-08-11 Thread GitBox


pirateskipper opened a new pull request #3248:
URL: https://github.com/apache/rocketmq/pull/3248


   **Make sure set the target branch to `develop`**
   
   ## What is the purpose of the change
   
   - Streamline code and increase readability
   
   ## Brief changelog
   
   - Safely delete parameters and variables that have never been used
   - Related change files:DefaultRequestProcessor && KVConfigManager
   
   ## Verifying this change
   
   - Verified


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-externals] liumaomaoqwer opened a new pull request #778: [ISSUE #777]change wrong variable name clinetId to clientId

2021-08-11 Thread GitBox


liumaomaoqwer opened a new pull request #778:
URL: https://github.com/apache/rocketmq-externals/pull/778


   ## What is the purpose of the change
   
   change wrong variable name clinetId to clientId
   
   ## Brief changelog
   
   The variable clinetId in Class ConsumerServiceImpl and MqttSampleConsumer is 
not correct, so i changed clinetId to clientId.
   
   ## Verifying this change
   
   
   
   Follow this checklist to help us incorporate your contribution quickly and 
easily. Notice, `it would be helpful if you could finish the following 5 
checklist(the last one is not necessary)before request the community to review 
your PR`.
   
   - [x] Make sure there is a [Github 
issue](https://github.com/apache/rocketmq/issues) filed for the change (usually 
before you start working on it). Trivial changes like typos do not require a 
Github issue. Your pull request should address just this issue, without pulling 
in other changes - one PR resolves one issue. 
   - [x] Format the pull request title like `[ISSUE #123] Fix UnknownException 
when host config not exist`. Each commit in the pull request should have a 
meaningful subject line and body.
   - [x] Write a pull request description that is detailed enough to understand 
what the pull request does, how, and why.
   - [x] Write necessary unit-test(over 80% coverage) to verify your logic 
correction, more mock a little better when cross module dependency exist. If 
the new feature or significant change is committed, please remember to add 
integration-test in [test 
module](https://github.com/apache/rocketmq/tree/master/test).
   - [x] Run `mvn -B clean apache-rat:check findbugs:findbugs 
checkstyle:checkstyle` to make sure basic checks pass. Run `mvn clean install 
-DskipITs` to make sure unit-test pass. Run `mvn clean test-compile 
failsafe:integration-test`  to make sure integration-test pass.
   - [ ] If this contribution is large, please file an [Apache Individual 
Contributor License Agreement](http://www.apache.org/licenses/#clas).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] coveralls commented on pull request #3248: Safely delete parameters and variables that have never been used

2021-08-11 Thread GitBox


coveralls commented on pull request #3248:
URL: https://github.com/apache/rocketmq/pull/3248#issuecomment-896714408


   
   [![Coverage 
Status](https://coveralls.io/builds/42060241/badge)](https://coveralls.io/builds/42060241)
   
   Coverage increased (+0.1%) to 54.212% when pulling 
**e52fc184773b63d343cfbbfc1342eb5dd1d75d72 on pirateskipper:r-issues** into 
**44bdeedad2d0e28717f5012d375d12af76305bb5 on apache:develop**.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] codecov-commenter commented on pull request #3248: Safely delete parameters and variables that have never been used

2021-08-11 Thread GitBox


codecov-commenter commented on pull request #3248:
URL: https://github.com/apache/rocketmq/pull/3248#issuecomment-896714532


   # 
[Codecov](https://codecov.io/gh/apache/rocketmq/pull/3248?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 Report
   > Merging 
[#3248](https://codecov.io/gh/apache/rocketmq/pull/3248?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 (e52fc18) into 
[develop](https://codecov.io/gh/apache/rocketmq/commit/5203d52f1bab431ada236ba279c80e62778f8a15?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 (5203d52) will **increase** coverage by `1.79%`.
   > The diff coverage is `31.25%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/rocketmq/pull/3248/graphs/tree.svg?width=650&height=150&src=pr&token=4w0sxP1wZv&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)](https://codecov.io/gh/apache/rocketmq/pull/3248?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   
   ```diff
   @@  Coverage Diff  @@
   ## develop#3248  +/-   ##
   =
   + Coverage  47.93%   49.72%   +1.79% 
   - Complexity  4564 5012 +448 
   =
 Files552  552  
 Lines  3662840169+3541 
 Branches4844 5813 +969 
   =
   + Hits   1755819975+2417 
   - Misses 1685017815 +965 
   - Partials2220 2379 +159 
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/rocketmq/pull/3248?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 | Coverage Δ | |
   |---|---|---|
   | 
[...tmq/namesrv/processor/DefaultRequestProcessor.java](https://codecov.io/gh/apache/rocketmq/pull/3248/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-bmFtZXNydi9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcm9ja2V0bXEvbmFtZXNydi9wcm9jZXNzb3IvRGVmYXVsdFJlcXVlc3RQcm9jZXNzb3IuamF2YQ==)
 | `35.93% <26.66%> (ø)` | |
   | 
[...che/rocketmq/namesrv/kvconfig/KVConfigManager.java](https://codecov.io/gh/apache/rocketmq/pull/3248/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-bmFtZXNydi9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcm9ja2V0bXEvbmFtZXNydi9rdmNvbmZpZy9LVkNvbmZpZ01hbmFnZXIuamF2YQ==)
 | `59.18% <100.00%> (ø)` | |
   | 
[.../rocketmq/broker/pagecache/OneMessageTransfer.java](https://codecov.io/gh/apache/rocketmq/pull/3248/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-YnJva2VyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9yb2NrZXRtcS9icm9rZXIvcGFnZWNhY2hlL09uZU1lc3NhZ2VUcmFuc2Zlci5qYXZh)
 | `22.58% <0.00%> (-10.76%)` | :arrow_down: |
   | 
[...a/org/apache/rocketmq/logging/inner/SysLogger.java](https://codecov.io/gh/apache/rocketmq/pull/3248/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-bG9nZ2luZy9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcm9ja2V0bXEvbG9nZ2luZy9pbm5lci9TeXNMb2dnZXIuamF2YQ==)
 | `28.57% <0.00%> (-5.72%)` | :arrow_down: |
   | 
[...ketmq/client/impl/consumer/PullMessageService.java](https://codecov.io/gh/apache/rocketmq/pull/3248/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-Y2xpZW50L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9yb2NrZXRtcS9jbGllbnQvaW1wbC9jb25zdW1lci9QdWxsTWVzc2FnZVNlcnZpY2UuamF2YQ==)
 | `75.55% <0.00%> (-4.45%)` | :arrow_down: |
   | 
[...java/org/apache/rocketmq/logging/inner/Logger.java](https://codecov.io/gh/apache/rocketmq/pull/3248/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-bG9nZ2luZy9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcm9ja2V0bXEvbG9nZ2luZy9pbm5lci9Mb2dnZXIuamF2YQ==)
 | `52.15% <0.00%> (-3.83%)` | :arrow_down: |
   | 
[...ava/org/apache/rocketmq/test/util/VerifyUtils.java](https://codecov.io/gh/apache/rocketmq/pull/3248/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-dGVzdC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcm9ja2V0bX

[GitHub] [rocketmq-client-cpp] ThreeAndTwo opened a new issue #366: EventLoop.h:112:15: error: field 'm_peerAddrPort' has incomplete type 'std::string'

2021-08-11 Thread GitBox


ThreeAndTwo opened a new issue #366:
URL: https://github.com/apache/rocketmq-client-cpp/issues/366


   relation issues: 
   - https://github.com/apache/rocketmq-client-cpp/issues/355
   - https://github.com/apache/rocketmq-client-cpp/issues/361
   
   when boost update to 1.72.0
   os: macos
   
![image](https://user-images.githubusercontent.com/35927054/129045121-b5045074-afb8-4b6d-afc8-a62408b6c483.png)
   
   if use default boost, there will happen 
[355](https://github.com/apache/rocketmq-client-cpp/issues/355)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] coveralls commented on pull request #3247: Safely delete parameters and variables that have never been used

2021-08-11 Thread GitBox


coveralls commented on pull request #3247:
URL: https://github.com/apache/rocketmq/pull/3247#issuecomment-896979957


   
   [![Coverage 
Status](https://coveralls.io/builds/42059840/badge)](https://coveralls.io/builds/42059840)
   
   Coverage decreased (-0.1%) to 53.992% when pulling 
**e52fc184773b63d343cfbbfc1342eb5dd1d75d72 on pirateskipper:r-issues** into 
**5203d52f1bab431ada236ba279c80e62778f8a15 on apache:release-4.9.0**.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-externals] StyleTang merged pull request #777: [ISSUE #673]Add unit tests to improve code coverage

2021-08-11 Thread GitBox


StyleTang merged pull request #777:
URL: https://github.com/apache/rocketmq-externals/pull/777


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-site] StyleTang merged pull request #90: Add new committer styletang

2021-08-11 Thread GitBox


StyleTang merged pull request #90:
URL: https://github.com/apache/rocketmq-site/pull/90


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-spring] zongtanghu commented on issue #297: Not eligible for auto-proxying info

2021-08-11 Thread GitBox


zongtanghu commented on issue #297:
URL: https://github.com/apache/rocketmq-spring/issues/297#issuecomment-897274352


   @zhangjidi2016 can you submit a pr to fix this issues?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-spring] zongtanghu closed issue #371: Add some asynchronous API for batch messages in RocketMQTemplate.

2021-08-11 Thread GitBox


zongtanghu closed issue #371:
URL: https://github.com/apache/rocketmq-spring/issues/371


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-spring] zongtanghu commented on issue #371: Add some asynchronous API for batch messages in RocketMQTemplate.

2021-08-11 Thread GitBox


zongtanghu commented on issue #371:
URL: https://github.com/apache/rocketmq-spring/issues/371#issuecomment-897274950


   The related pr has already been merged, so I closed this issues!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-spring] vongosling commented on issue #371: Add some asynchronous API for batch messages in RocketMQTemplate.

2021-08-11 Thread GitBox


vongosling commented on issue #371:
URL: https://github.com/apache/rocketmq-spring/issues/371#issuecomment-897276732


   @zongtanghu you could not close pr by yourself. just link the pr with the 
issue. when merging to the main branch, it will automatically close.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-spring] vongosling commented on pull request #372: [ISSUE #371] Add some asynchronous API for batch messages in RocketMQ…

2021-08-11 Thread GitBox


vongosling commented on pull request #372:
URL: https://github.com/apache/rocketmq-spring/pull/372#issuecomment-897276923


   Linked issues @zongtanghu 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-spring] zongtanghu commented on issue #371: Add some asynchronous API for batch messages in RocketMQTemplate.

2021-08-11 Thread GitBox


zongtanghu commented on issue #371:
URL: https://github.com/apache/rocketmq-spring/issues/371#issuecomment-897278311


   ok!
   
   
原始邮件 
   发件人: von ***@***.***>
   收件人: ***@***.***>
   抄送: Hu ***@***.***>; ***@***.***>
   发送时间: 2021年8月12日(周四) 09:30
   主题: Re: [apache/rocketmq-spring] Add some asynchronous API for batchmessages 
in RocketMQTemplate. (#371)
   
   
   @zongtanghu you could not close pr by yourself. just link the pr with the 
issue. when merging to the main branch, it will automatically close.
   —
   You are receiving this because you were mentioned.
   Reply to this email directly, view it on GitHub, or unsubscribe.
   Triage notifications on the go with GitHub Mobile for iOS or Android.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-site] vongosling opened a new issue #91: Remove issues colunm in contact page

2021-08-11 Thread GitBox


vongosling opened a new issue #91:
URL: https://github.com/apache/rocketmq-site/issues/91


   Consider we have migrated from Jira to issues[1]. this mail list has lost 
its value. Anybody would like to remove it ?
   
   
   [1] https://rocketmq.apache.org/about/contact/


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] 88888-IT opened a new issue #3249: Why does rocketMq store data on a file system instead of Redis

2021-08-11 Thread GitBox


8-IT opened a new issue #3249:
URL: https://github.com/apache/rocketmq/issues/3249


   Why does rocketMq store data on a file system instead of Redis


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] 88888-IT opened a new issue #3250: RocketMq is currently designed to have one or more consumequeues for each topic. Why not one or more consumequeues for each topic and tag

2021-08-11 Thread GitBox


8-IT opened a new issue #3250:
URL: https://github.com/apache/rocketmq/issues/3250


   RocketMq is currently designed to have one or more consumequeues for each 
topic. Why not one or more consumequeues for each topic and tag


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-client-go] zongtanghu commented on issue #696: pull consumer has not supported

2021-08-11 Thread GitBox


zongtanghu commented on issue #696:
URL: 
https://github.com/apache/rocketmq-client-go/issues/696#issuecomment-897289169


   I think you can write an design document, and then I and my workmates can 
participate into coding.And then we can splite this issues into serveral prs. 
@wenfengwang 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] ni-ze commented on issue #3250: RocketMq is currently designed to have one or more consumequeues for each topic. Why not one or more consumequeues for each topic and tag

2021-08-11 Thread GitBox


ni-ze commented on issue #3250:
URL: https://github.com/apache/rocketmq/issues/3250#issuecomment-897299435


   tag is decided by use, broker can not control how many tags users use. if we 
design consume queues by tag, that is a disaster for broker stability.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] ni-ze edited a comment on issue #3250: RocketMq is currently designed to have one or more consumequeues for each topic. Why not one or more consumequeues for each topic and tag

2021-08-11 Thread GitBox


ni-ze edited a comment on issue #3250:
URL: https://github.com/apache/rocketmq/issues/3250#issuecomment-897299435


   tag is decided by user, broker can not control how many tags users use. if 
we design consume queues by tag, that is a disaster for broker stability.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] 88888-IT commented on issue #3250: RocketMq is currently designed to have one or more consumequeues for each topic. Why not one or more consumequeues for each topic and tag

2021-08-11 Thread GitBox


8-IT commented on issue #3250:
URL: https://github.com/apache/rocketmq/issues/3250#issuecomment-897303372


   > tag is decided by user, broker can not control how many tags users use. if 
we design consume queues by tag, that is a disaster for broker stability.
   
   What if the user defines the tag and creates it in the broker


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-site] yuz10 opened a new pull request #92: [ISSUE #91] remove issues from mailing list

2021-08-11 Thread GitBox


yuz10 opened a new pull request #92:
URL: https://github.com/apache/rocketmq-site/pull/92


   Please do not create a Pull Request without creating an issue first. 
   
   ## What is the purpose of the change
   
   #91
   
   ## Brief changelog
   
   XX
   
   ## Verifying this change
   
   
   
   Follow this checklist to help us incorporate your contribution quickly and 
easily:
   
   - [x] Make sure there is a Github issue filed for the change (usually before 
you start working on it). Trivial changes like typos do not require a Github 
issue. Your pull request should address just this issue, without pulling in 
other changes - one PR resolves one issue. 
   - [ ] Format the pull request title like `[ISSUE #123] Fix UnknownException 
when host config not exist`. Each commit in the pull request should have a 
meaningful subject line and body.
   - [ ] Write a pull request description that is detailed enough to understand 
what the pull request does, how, and why.
   - [ ] Write necessary unit-test to verify your logic correction, more mock a 
little better when cross module dependency exist. If the new feature or 
significant change is committed, please remember to add integration-test in 
[test module](https://github.com/apache/rocketmq/tree/master/test).
   - [ ] Run `mvn -B clean apache-rat:check findbugs:findbugs 
checkstyle:checkstyle` to make sure basic checks pass. Run `mvn clean install 
-DskipITs` to make sure unit-test pass. Run `mvn clean test-compile 
failsafe:integration-test`  to make sure integration-test pass.
   - [ ] If this contribution is large, please file an [Apache Individual 
Contributor License Agreement](http://www.apache.org/licenses/#clas).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] novayoung opened a new issue #3251: Does the lower version of rocketmq client support failover?

2021-08-11 Thread GitBox


novayoung opened a new issue #3251:
URL: https://github.com/apache/rocketmq/issues/3251


   My client version is 4.5.1,but server version is 4.6


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-client-go] yuz10 merged pull request #709: [ISSUE #710] Fix send duplicate message in trace interceptor

2021-08-11 Thread GitBox


yuz10 merged pull request #709:
URL: https://github.com/apache/rocketmq-client-go/pull/709


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] coveralls edited a comment on pull request #3149: [ISSUE #3148]Support metadata export

2021-08-11 Thread GitBox


coveralls edited a comment on pull request #3149:
URL: https://github.com/apache/rocketmq/pull/3149#issuecomment-880750487


   
   [![Coverage 
Status](https://coveralls.io/builds/42085950/badge)](https://coveralls.io/builds/42085950)
   
   Coverage increased (+0.02%) to 54.195% when pulling 
**064a3b80e6c4fcdbb611ac376dc59a5f2c8537e9 on panzhi33:develop** into 
**ec985ee67354a55607c3b5fde2e110dfb1268fea on apache:develop**.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] codecov-commenter edited a comment on pull request #3149: [ISSUE #3148]Support metadata export

2021-08-11 Thread GitBox


codecov-commenter edited a comment on pull request #3149:
URL: https://github.com/apache/rocketmq/pull/3149#issuecomment-889850553


   # 
[Codecov](https://codecov.io/gh/apache/rocketmq/pull/3149?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 Report
   > Merging 
[#3149](https://codecov.io/gh/apache/rocketmq/pull/3149?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 (064a3b8) into 
[develop](https://codecov.io/gh/apache/rocketmq/commit/30058cffa5a2512894a61b5c5eb97a58ce6681ec?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 (30058cf) will **increase** coverage by `0.11%`.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/rocketmq/pull/3149/graphs/tree.svg?width=650&height=150&src=pr&token=4w0sxP1wZv&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)](https://codecov.io/gh/apache/rocketmq/pull/3149?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   
   ```diff
   @@  Coverage Diff  @@
   ## develop#3149  +/-   ##
   =
   + Coverage  47.82%   47.94%   +0.11% 
   + Complexity  4551 4333 -218 
   =
 Files552  514  -38 
 Lines  3664334972-1671 
 Branches4846 4661 -185 
   =
   - Hits   1752616768 -758 
   + Misses 1688116026 -855 
   + Partials2236 2178  -58 
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/rocketmq/pull/3149?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 | Coverage Δ | |
   |---|---|---|
   | 
[...rocketmq/broker/pagecache/ManyMessageTransfer.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-YnJva2VyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9yb2NrZXRtcS9icm9rZXIvcGFnZWNhY2hlL01hbnlNZXNzYWdlVHJhbnNmZXIuamF2YQ==)
 | `36.36% <0.00%> (-9.80%)` | :arrow_down: |
   | 
[.../rocketmq/broker/pagecache/OneMessageTransfer.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-YnJva2VyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9yb2NrZXRtcS9icm9rZXIvcGFnZWNhY2hlL09uZU1lc3NhZ2VUcmFuc2Zlci5qYXZh)
 | `24.00% <0.00%> (-9.34%)` | :arrow_down: |
   | 
[...ketmq/common/protocol/body/ConsumerConnection.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9yb2NrZXRtcS9jb21tb24vcHJvdG9jb2wvYm9keS9Db25zdW1lckNvbm5lY3Rpb24uamF2YQ==)
 | `95.83% <0.00%> (-4.17%)` | :arrow_down: |
   | 
[...rg/apache/rocketmq/remoting/netty/NettyLogger.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-cmVtb3Rpbmcvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3JvY2tldG1xL3JlbW90aW5nL25ldHR5L05ldHR5TG9nZ2VyLmphdmE=)
 | `16.32% <0.00%> (-2.73%)` | :arrow_down: |
   | 
[...he/rocketmq/client/trace/AsyncTraceDispatcher.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-Y2xpZW50L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9yb2NrZXRtcS9jbGllbnQvdHJhY2UvQXN5bmNUcmFjZURpc3BhdGNoZXIuamF2YQ==)
 | `77.72% <0.00%> (-1.49%)` | :arrow_down: |
   | 
[...pl/consumer/ConsumeMessageConcurrentlyService.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-Y2xpZW50L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9yb2NrZXRtcS9jbGllbnQvaW1wbC9jb25zdW1lci9Db25zdW1lTWVzc2FnZUNvbmN1cnJlbnRseVNlcnZpY2UuamF2YQ==)
 | `47.57% <0.00%> (-0.98%)` | :arrow_down: |
   | 
[...he/rocketmq/client/impl/consumer/ProcessQueue.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache

[GitHub] [rocketmq] zhxjdwh opened a new issue #3252: the rocketmq_client.log file spent to mush cache/buffer

2021-08-11 Thread GitBox


zhxjdwh opened a new issue #3252:
URL: https://github.com/apache/rocketmq/issues/3252


   
   **BUG REPORT**
   the rocketmq_client.log file spent to mush cache/buffer !
   These log files really consume too much cache/buffer. The jvm memory usage 
of my container is only 500M, but the log buffer/cache of Rocketmq consumes 2G 
of memory.
   
这几个日志文件实在消耗了太多cache/buffer了,我容器的jvm内存使用才500M,但是rocketmq的日志buffer/cache就消耗了2G的内存
   ```
   
   [root@node135 
f914151fc9ae5ce5f477a8d0d69ed9b0ebbb2ea45be3aec0f0acbde859d3e3b1]# 
/usr/local/bin/vmtouch ./diff/root/logs/rocketmqlogs/**rocketmq_client.log**
  Files: 1
Directories: 0
 **Resident Pages: 198301/198301  774M/774M  100%**
Elapsed: 0.03113 seconds
   [root@node135 
f914151fc9ae5ce5f477a8d0d69ed9b0ebbb2ea45be3aec0f0acbde859d3e3b1]# 
/usr/local/bin/vmtouch ./diff/root/logs/rocketmqlogs/**rocketmq_client.log.1**
  Files: 1
Directories: 0
 **Resident Pages: 262145/262145  1G/1G  100%**
Elapsed: 0.042067 seconds
   [root@node135 
f914151fc9ae5ce5f477a8d0d69ed9b0ebbb2ea45be3aec0f0acbde859d3e3b1]# 
/usr/local/bin/vmtouch **./diff/root/logs/rocketmqlogs/rocketmq_client.log.2**
  Files: 1
Directories: 0
 **Resident Pages: 120207/262145  469M/1G  45.9%**
Elapsed: 0.020819 seconds
   ```
   
   **FEATURE REQUEST**
   Reduce the buffer/cache memory usage of log files
   减少log文件的buffer/cache内存占用
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] novayoung closed issue #3251: Does the lower version of rocketmq client support failover?

2021-08-11 Thread GitBox


novayoung closed issue #3251:
URL: https://github.com/apache/rocketmq/issues/3251


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] coveralls edited a comment on pull request #3149: [ISSUE #3148]Support metadata export

2021-08-11 Thread GitBox


coveralls edited a comment on pull request #3149:
URL: https://github.com/apache/rocketmq/pull/3149#issuecomment-880750487


   
   [![Coverage 
Status](https://coveralls.io/builds/42086837/badge)](https://coveralls.io/builds/42086837)
   
   Coverage decreased (-0.03%) to 54.146% when pulling 
**b61009195c85d407e2c1b8fc6d0691a0be938b1a on panzhi33:develop** into 
**ec985ee67354a55607c3b5fde2e110dfb1268fea on apache:develop**.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] codecov-commenter edited a comment on pull request #3149: [ISSUE #3148]Support metadata export

2021-08-11 Thread GitBox


codecov-commenter edited a comment on pull request #3149:
URL: https://github.com/apache/rocketmq/pull/3149#issuecomment-889850553


   # 
[Codecov](https://codecov.io/gh/apache/rocketmq/pull/3149?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 Report
   > Merging 
[#3149](https://codecov.io/gh/apache/rocketmq/pull/3149?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 (b610091) into 
[develop](https://codecov.io/gh/apache/rocketmq/commit/30058cffa5a2512894a61b5c5eb97a58ce6681ec?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 (30058cf) will **increase** coverage by `0.05%`.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/rocketmq/pull/3149/graphs/tree.svg?width=650&height=150&src=pr&token=4w0sxP1wZv&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)](https://codecov.io/gh/apache/rocketmq/pull/3149?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   
   ```diff
   @@  Coverage Diff  @@
   ## develop#3149  +/-   ##
   =
   + Coverage  47.82%   47.88%   +0.05% 
   + Complexity  4551 4326 -225 
   =
 Files552  514  -38 
 Lines  3664334972-1671 
 Branches4846 4661 -185 
   =
   - Hits   1752616746 -780 
   + Misses 1688116048 -833 
   + Partials2236 2178  -58 
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/rocketmq/pull/3149?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 | Coverage Δ | |
   |---|---|---|
   | 
[...rocketmq/broker/pagecache/ManyMessageTransfer.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-YnJva2VyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9yb2NrZXRtcS9icm9rZXIvcGFnZWNhY2hlL01hbnlNZXNzYWdlVHJhbnNmZXIuamF2YQ==)
 | `36.36% <0.00%> (-9.80%)` | :arrow_down: |
   | 
[.../rocketmq/broker/pagecache/OneMessageTransfer.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-YnJva2VyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9yb2NrZXRtcS9icm9rZXIvcGFnZWNhY2hlL09uZU1lc3NhZ2VUcmFuc2Zlci5qYXZh)
 | `24.00% <0.00%> (-9.34%)` | :arrow_down: |
   | 
[...ketmq/common/protocol/body/ConsumerConnection.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9yb2NrZXRtcS9jb21tb24vcHJvdG9jb2wvYm9keS9Db25zdW1lckNvbm5lY3Rpb24uamF2YQ==)
 | `95.83% <0.00%> (-4.17%)` | :arrow_down: |
   | 
[...rg/apache/rocketmq/remoting/netty/NettyLogger.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-cmVtb3Rpbmcvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3JvY2tldG1xL3JlbW90aW5nL25ldHR5L05ldHR5TG9nZ2VyLmphdmE=)
 | `16.32% <0.00%> (-2.73%)` | :arrow_down: |
   | 
[...mq/client/impl/consumer/RebalanceLitePullImpl.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-Y2xpZW50L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9yb2NrZXRtcS9jbGllbnQvaW1wbC9jb25zdW1lci9SZWJhbGFuY2VMaXRlUHVsbEltcGwuamF2YQ==)
 | `48.52% <0.00%> (-2.22%)` | :arrow_down: |
   | 
[...he/rocketmq/client/trace/AsyncTraceDispatcher.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-Y2xpZW50L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9yb2NrZXRtcS9jbGllbnQvdHJhY2UvQXN5bmNUcmFjZURpc3BhdGNoZXIuamF2YQ==)
 | `77.72% <0.00%> (-1.49%)` | :arrow_down: |
   | 
[...he/rocketmq/client/impl/consumer/ProcessQueue.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Founda

[GitHub] [rocketmq] ni-ze commented on issue #3252: the rocketmq_client.log file spent to mush cache/buffer

2021-08-11 Thread GitBox


ni-ze commented on issue #3252:
URL: https://github.com/apache/rocketmq/issues/3252#issuecomment-897371476


   can you give a way to repeat this? or just it will appear anyway?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] ni-ze commented on issue #3250: RocketMq is currently designed to have one or more consumequeues for each topic. Why not one or more consumequeues for each topic and tag

2021-08-11 Thread GitBox


ni-ze commented on issue #3250:
URL: https://github.com/apache/rocketmq/issues/3250#issuecomment-897372515


   > > tag is decided by user, broker can not control how many tags users use. 
if we design consume queues by tag, that is a disaster for broker stability.
   > 
   > What if the user defines the tag and creates it in the broker
   You can not do that. Tag is define by user when send a message  in producer.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] 88888-IT commented on issue #3250: RocketMq is currently designed to have one or more consumequeues for each topic. Why not one or more consumequeues for each topic and tag

2021-08-11 Thread GitBox


8-IT commented on issue #3250:
URL: https://github.com/apache/rocketmq/issues/3250#issuecomment-897375677


   > What if the user defines the tag and creates it in the broker
   > You can not do that. Tag is define by user when send a message in producer.
   
   我的意思是如果生产者提前在broker中创建好tag再去使用,这个时候是不是就可以根据tag来区分consumequeue


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] odbozhou commented on issue #3249: Why does rocketMq store data on a file system instead of Redis

2021-08-11 Thread GitBox


odbozhou commented on issue #3249:
URL: https://github.com/apache/rocketmq/issues/3249#issuecomment-897376226


   What is the problem with the file system? Why should it be stored in redis? 
Please describe your problem in detail.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] ni-ze commented on issue #3250: RocketMq is currently designed to have one or more consumequeues for each topic. Why not one or more consumequeues for each topic and tag

2021-08-11 Thread GitBox


ni-ze commented on issue #3250:
URL: https://github.com/apache/rocketmq/issues/3250#issuecomment-897381697


   why you need distinguish consumer queue by tag? It is topic dimension right 
now.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] duhenglucky closed issue #3249: Why does rocketMq store data on a file system instead of Redis

2021-08-11 Thread GitBox


duhenglucky closed issue #3249:
URL: https://github.com/apache/rocketmq/issues/3249


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] zhxjdwh commented on issue #3252: the rocketmq_client.log file spent to mush cache/buffer

2021-08-12 Thread GitBox


zhxjdwh commented on issue #3252:
URL: https://github.com/apache/rocketmq/issues/3252#issuecomment-897402857


   @ni-ze 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] zhxjdwh removed a comment on issue #3252: the rocketmq_client.log file spent to mush cache/buffer

2021-08-12 Thread GitBox


zhxjdwh removed a comment on issue #3252:
URL: https://github.com/apache/rocketmq/issues/3252#issuecomment-897402857


   @ni-ze 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] 88888-IT commented on issue #3250: RocketMq is currently designed to have one or more consumequeues for each topic. Why not one or more consumequeues for each topic and tag

2021-08-12 Thread GitBox


8-IT commented on issue #3250:
URL: https://github.com/apache/rocketmq/issues/3250#issuecomment-897408116


   > why you need distinguish consumer queue by tag? It is topic dimension 
right now.
   
   If the consumer subscribes to the tag under the topic, the consumer needs to 
traverse the queue and compare the tagHashCode to fetch the item


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-client-go] vongosling closed issue #710: [Master] message will be send twice in trace interceptor

2021-08-12 Thread GitBox


vongosling closed issue #710:
URL: https://github.com/apache/rocketmq-client-go/issues/710


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-client-go] maqingxiang commented on issue #671: get subscribed topic list for PushConsumer

2021-08-12 Thread GitBox


maqingxiang commented on issue #671:
URL: 
https://github.com/apache/rocketmq-client-go/issues/671#issuecomment-897427487


   The business only needs to provide the latest topic subscription list, and 
then the platform can dynamically adjust the topic subscription list according 
to the latest requirements of the business.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-externals] SteNicholas opened a new pull request #779: [#715] Support the RocketMQ TableSource based on the legacy Source implementation

2021-08-12 Thread GitBox


SteNicholas opened a new pull request #779:
URL: https://github.com/apache/rocketmq-externals/pull/779


   ## What is the purpose of the change
   
   At present, RocketMQ connector supports the implementation of legacy 
SourceFunction interface, which needs the `ScanTableSource` implementation to 
expand Table/SQL capabilities. RocketMQ connector should support the RocketMQ 
TableSource based on the legacy SourceFunction interface.
   
   ## Brief changelog
   
- Support `RocketMQScanTableSource` based on the implementation 
`RocketMQSourceFunction` of legacy `SourceFunction` interface.
   
   ## Verifying this change
   
   Follow this checklist to help us incorporate your contribution quickly and 
easily. Notice, `it would be helpful if you could finish the following 5 
checklist(the last one is not necessary)before request the community to review 
your PR`.
   
   - [x] Make sure there is a [Github 
issue](https://github.com/apache/rocketmq/issues) filed for the change (usually 
before you start working on it). Trivial changes like typos do not require a 
Github issue. Your pull request should address just this issue, without pulling 
in other changes - one PR resolves one issue. 
   - [x] Format the pull request title like `[ISSUE #123] Fix UnknownException 
when host config not exist`. Each commit in the pull request should have a 
meaningful subject line and body.
   - [x] Write a pull request description that is detailed enough to understand 
what the pull request does, how, and why.
   - [x] Write necessary unit-test(over 80% coverage) to verify your logic 
correction, more mock a little better when cross module dependency exist. If 
the new feature or significant change is committed, please remember to add 
integration-test in [test 
module](https://github.com/apache/rocketmq/tree/master/test).
   - [x] Run `mvn -B clean apache-rat:check findbugs:findbugs 
checkstyle:checkstyle` to make sure basic checks pass. Run `mvn clean install 
-DskipITs` to make sure unit-test pass. Run `mvn clean test-compile 
failsafe:integration-test`  to make sure integration-test pass.
   - [ ] If this contribution is large, please file an [Apache Individual 
Contributor License Agreement](http://www.apache.org/licenses/#clas).


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-client-go] maqingxiang opened a new issue #711: [bug] The Unsubscribe() interface influence retrytopic

2021-08-12 Thread GitBox


maqingxiang opened a new issue #711:
URL: https://github.com/apache/rocketmq-client-go/issues/711


   BUG REPORT
   
   // push_consumer.go
   
   func (pc *DefaultPushConsumer) Unsubscribe(topic string) error {
   ...
   retryTopic := internal.GetRetryTopic(pc.consumerGroup)
   pc.subscriptionDataTable.Delete(retryTopic)
   ...
   }
   
   There's a scene where the same ConsumerGroup subscries to T1, subscries to 
T2, and unsubscribe T1. Is that a problem?
   
   the client version is github.com/apache/rocketmq-client-go/v2 v2.1.0


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-site] vongosling merged pull request #84: Use .asf.yaml for publishing

2021-08-12 Thread GitBox


vongosling merged pull request #84:
URL: https://github.com/apache/rocketmq-site/pull/84


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] zhaohai1299002788 commented on issue #3252: the rocketmq_client.log file spent too much cache/buffer memory

2021-08-12 Thread GitBox


zhaohai1299002788 commented on issue #3252:
URL: https://github.com/apache/rocketmq/issues/3252#issuecomment-897436932


   If it is a log file, you can write a timer to delete the old log file.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-site] yuz10 opened a new pull request #93: Add new committer yuz10

2021-08-12 Thread GitBox


yuz10 opened a new pull request #93:
URL: https://github.com/apache/rocketmq-site/pull/93


   Please do not create a Pull Request without creating an issue first. 
   
   ## What is the purpose of the change
   
   X
   
   ## Brief changelog
   
   XX
   
   ## Verifying this change
   
   
   
   Follow this checklist to help us incorporate your contribution quickly and 
easily:
   
   - [x] Make sure there is a Github issue filed for the change (usually before 
you start working on it). Trivial changes like typos do not require a Github 
issue. Your pull request should address just this issue, without pulling in 
other changes - one PR resolves one issue. 
   - [ ] Format the pull request title like `[ISSUE #123] Fix UnknownException 
when host config not exist`. Each commit in the pull request should have a 
meaningful subject line and body.
   - [ ] Write a pull request description that is detailed enough to understand 
what the pull request does, how, and why.
   - [ ] Write necessary unit-test to verify your logic correction, more mock a 
little better when cross module dependency exist. If the new feature or 
significant change is committed, please remember to add integration-test in 
[test module](https://github.com/apache/rocketmq/tree/master/test).
   - [ ] Run `mvn -B clean apache-rat:check findbugs:findbugs 
checkstyle:checkstyle` to make sure basic checks pass. Run `mvn clean install 
-DskipITs` to make sure unit-test pass. Run `mvn clean test-compile 
failsafe:integration-test`  to make sure integration-test pass.
   - [ ] If this contribution is large, please file an [Apache Individual 
Contributor License Agreement](http://www.apache.org/licenses/#clas).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-client-go] maixiaohai commented on issue #711: [bug] The Unsubscribe() interface influence retrytopic

2021-08-12 Thread GitBox


maixiaohai commented on issue #711:
URL: 
https://github.com/apache/rocketmq-client-go/issues/711#issuecomment-897458107


   @maqingxiang agree with you, welcome to submit a pr to fix it.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-client-go] maixiaohai merged pull request #706: [ISSUE #595] Update go.mod

2021-08-12 Thread GitBox


maixiaohai merged pull request #706:
URL: https://github.com/apache/rocketmq-client-go/pull/706


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-client-go] maixiaohai closed issue #595: There is a vulnerability in gjson v1.2.1,upgrade recommended

2021-08-12 Thread GitBox


maixiaohai closed issue #595:
URL: https://github.com/apache/rocketmq-client-go/issues/595


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] kingmax-yu opened a new issue #3253: RockerMQ-4.7.1 storeerror.log reprot too many errorlog "WARN FlushConsumeQueueService - Offset not matched"

2021-08-12 Thread GitBox


kingmax-yu opened a new issue #3253:
URL: https://github.com/apache/rocketmq/issues/3253


   WARN FlushConsumeQueueService - Offset not matched. Request offset: 
31800, firstOffset: 31200, lastOffset: 31800, mappedFileSize: 
600, mappedFiles count: 1


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] lizhiboo commented on issue #3243: use the StringBuilder another method can reduce impact on array copy performance

2021-08-12 Thread GitBox


lizhiboo commented on issue #3243:
URL: https://github.com/apache/rocketmq/issues/3243#issuecomment-897463732


   @gylang  Good catch, can u submit a pr to optimize it?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] lizhiboo commented on issue #3253: RockerMQ-4.7.1 storeerror.log reprot too many errorlog "WARN FlushConsumeQueueService - Offset not matched"

2021-08-12 Thread GitBox


lizhiboo commented on issue #3253:
URL: https://github.com/apache/rocketmq/issues/3253#issuecomment-897477077


   > WARN FlushConsumeQueueService - Offset not matched. Request offset: 
31800, firstOffset: 31200, lastOffset: 31800, mappedFileSize: 
600, mappedFiles count: 1
   
   It seems that no new message produced, do not find message by Request offset.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] coveralls edited a comment on pull request #3149: [ISSUE #3148]Support metadata export

2021-08-12 Thread GitBox


coveralls edited a comment on pull request #3149:
URL: https://github.com/apache/rocketmq/pull/3149#issuecomment-880750487


   
   [![Coverage 
Status](https://coveralls.io/builds/42094499/badge)](https://coveralls.io/builds/42094499)
   
   Coverage decreased (-0.04%) to 54.129% when pulling 
**d47f051a56bc21b29663c9fe9bdbc031e9fad3b0 on panzhi33:develop** into 
**ec985ee67354a55607c3b5fde2e110dfb1268fea on apache:develop**.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] codecov-commenter edited a comment on pull request #3149: [ISSUE #3148]Support metadata export

2021-08-12 Thread GitBox


codecov-commenter edited a comment on pull request #3149:
URL: https://github.com/apache/rocketmq/pull/3149#issuecomment-889850553


   # 
[Codecov](https://codecov.io/gh/apache/rocketmq/pull/3149?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 Report
   > Merging 
[#3149](https://codecov.io/gh/apache/rocketmq/pull/3149?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 (d47f051) into 
[develop](https://codecov.io/gh/apache/rocketmq/commit/30058cffa5a2512894a61b5c5eb97a58ce6681ec?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 (30058cf) will **increase** coverage by `0.12%`.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/rocketmq/pull/3149/graphs/tree.svg?width=650&height=150&src=pr&token=4w0sxP1wZv&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)](https://codecov.io/gh/apache/rocketmq/pull/3149?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   
   ```diff
   @@  Coverage Diff  @@
   ## develop#3149  +/-   ##
   =
   + Coverage  47.82%   47.95%   +0.12% 
   + Complexity  4551 4336 -215 
   =
 Files552  514  -38 
 Lines  3664334972-1671 
 Branches4846 4661 -185 
   =
   - Hits   1752616771 -755 
   + Misses 1688116026 -855 
   + Partials2236 2175  -61 
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/rocketmq/pull/3149?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 | Coverage Δ | |
   |---|---|---|
   | 
[...rocketmq/broker/pagecache/ManyMessageTransfer.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-YnJva2VyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9yb2NrZXRtcS9icm9rZXIvcGFnZWNhY2hlL01hbnlNZXNzYWdlVHJhbnNmZXIuamF2YQ==)
 | `36.36% <0.00%> (-9.80%)` | :arrow_down: |
   | 
[.../rocketmq/broker/pagecache/OneMessageTransfer.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-YnJva2VyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9yb2NrZXRtcS9icm9rZXIvcGFnZWNhY2hlL09uZU1lc3NhZ2VUcmFuc2Zlci5qYXZh)
 | `24.00% <0.00%> (-9.34%)` | :arrow_down: |
   | 
[...ketmq/common/protocol/body/ConsumerConnection.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9yb2NrZXRtcS9jb21tb24vcHJvdG9jb2wvYm9keS9Db25zdW1lckNvbm5lY3Rpb24uamF2YQ==)
 | `95.83% <0.00%> (-4.17%)` | :arrow_down: |
   | 
[...rg/apache/rocketmq/remoting/netty/NettyLogger.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-cmVtb3Rpbmcvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3JvY2tldG1xL3JlbW90aW5nL25ldHR5L05ldHR5TG9nZ2VyLmphdmE=)
 | `16.32% <0.00%> (-2.73%)` | :arrow_down: |
   | 
[...pl/consumer/ConsumeMessageConcurrentlyService.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-Y2xpZW50L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9yb2NrZXRtcS9jbGllbnQvaW1wbC9jb25zdW1lci9Db25zdW1lTWVzc2FnZUNvbmN1cnJlbnRseVNlcnZpY2UuamF2YQ==)
 | `47.57% <0.00%> (-0.98%)` | :arrow_down: |
   | 
[...he/rocketmq/client/impl/consumer/ProcessQueue.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-Y2xpZW50L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9yb2NrZXRtcS9jbGllbnQvaW1wbC9jb25zdW1lci9Qcm9jZXNzUXVldWUuamF2YQ==)
 | `59.06% <0.00%> (-0.94%)` | :arrow_down: |
   | 
[...ocketmq/broker/processor/SendMessageProcessor.java](https://codecov.io/gh/apache/rocketmq/pull/3149/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache

[GitHub] [rocketmq] ni-ze opened a new pull request #3254: add new allocate MessageQueue strategy

2021-08-12 Thread GitBox


ni-ze opened a new pull request #3254:
URL: https://github.com/apache/rocketmq/pull/3254


   add new strategy: AllocateMessageQueueByMachine
   
   allocate message queue to client which local in the same machine 
preferentially, if the result is null, use the second strategy. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] ni-ze opened a new issue #3255: add new allocate MessageQueue strategy

2021-08-12 Thread GitBox


ni-ze opened a new issue #3255:
URL: https://github.com/apache/rocketmq/issues/3255


   add new strategy: AllocateMessageQueueByMachine
   
   allocate message queue to client which local in the same machine 
preferentially, if the result is null, use the second strategy.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] ni-ze removed a comment on issue #3252: the rocketmq_client.log file spent too much cache/buffer memory

2021-08-12 Thread GitBox


ni-ze removed a comment on issue #3252:
URL: https://github.com/apache/rocketmq/issues/3252#issuecomment-897371476


   can you give a way to repeat this? or just it will appear anyway?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] ni-ze commented on issue #3252: the rocketmq_client.log file spent too much cache/buffer memory

2021-08-12 Thread GitBox


ni-ze commented on issue #3252:
URL: https://github.com/apache/rocketmq/issues/3252#issuecomment-897586883


   here cache/buffer has nothing to do with your jvm, it manage by os use it to 
to accelerate read/write. It would not affect rocketmq client work. Just ignore 
it.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] ni-ze edited a comment on issue #3252: the rocketmq_client.log file spent too much cache/buffer memory

2021-08-12 Thread GitBox


ni-ze edited a comment on issue #3252:
URL: https://github.com/apache/rocketmq/issues/3252#issuecomment-897586883


   here cache/buffer has nothing to do with your jvm, it manage by os, use it 
to accelerate read/write. It would not affect rocketmq client work. Just ignore 
it.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-site] vongosling merged pull request #93: Add new committer yuz10

2021-08-12 Thread GitBox


vongosling merged pull request #93:
URL: https://github.com/apache/rocketmq-site/pull/93


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] schnell18 opened a new pull request #3256: Ignore irrelevant commit log files on startup

2021-08-12 Thread GitBox


schnell18 opened a new pull request #3256:
URL: https://github.com/apache/rocketmq/pull/3256


   - Prevent irrelevant files under $storePathCommitLog failing startup
   - Make RocketMQ easy to operate and bring less surprise to average 
administrators
   
   
   **Make sure set the target branch to `develop`**
   
   ## What is the purpose of the change
   
   Make RocketMQ more tolerant to minor administrator mistake and reduce 
service outage in case less experienced admin
   create irrelevant files under $storePathCommitLog directory.
   
   ## Brief changelog
   
   XX
   
   ## Verifying this change
   
   
   
   Follow this checklist to help us incorporate your contribution quickly and 
easily. Notice, `it would be helpful if you could finish the following 5 
checklist(the last one is not necessary)before request the community to review 
your PR`.
   
   - [x] Make sure there is a [Github 
issue](https://github.com/apache/rocketmq/issues) filed for the change (usually 
before you start working on it). Trivial changes like typos do not require a 
Github issue. Your pull request should address just this issue, without pulling 
in other changes - one PR resolves one issue. 
   - [x] Format the pull request title like `[ISSUE #123] Fix UnknownException 
when host config not exist`. Each commit in the pull request should have a 
meaningful subject line and body.
   - [x] Write a pull request description that is detailed enough to understand 
what the pull request does, how, and why.
   - [x] Write necessary unit-test(over 80% coverage) to verify your logic 
correction, more mock a little better when cross module dependency exist. If 
the new feature or significant change is committed, please remember to add 
integration-test in [test 
module](https://github.com/apache/rocketmq/tree/master/test).
   - [x] Run `mvn -B clean apache-rat:check findbugs:findbugs 
checkstyle:checkstyle` to make sure basic checks pass. Run `mvn clean install 
-DskipITs` to make sure unit-test pass. Run `mvn clean test-compile 
failsafe:integration-test`  to make sure integration-test pass.
   - [ ] If this contribution is large, please file an [Apache Individual 
Contributor License Agreement](http://www.apache.org/licenses/#clas).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] schnell18 opened a new issue #3257: Ignore irrelevant commit log files on startup

2021-08-12 Thread GitBox


schnell18 opened a new issue #3257:
URL: https://github.com/apache/rocketmq/issues/3257


   **BUG REPORT**
   
   1. Please describe the issue you observed:
   
   - What did you do?
   Create an empty file under $storePathCommitLog directory. Say, you have 
commit log store under /data0/rocketmq/commitlogs/,
   then you create a file named 'the-bad-empty-file-fails-rocketmq-startup".
   Restart broker.
   Then the broker will not be able to startup normal.
   You may see error message like "the-bad-empty-file-fails-rocketmq-startup 0 
length not matched message store value. please check it manually" in the 
storeerror.log.
   
   - What did you expect to see?
   Even in this case, RocketMQ broker should startup normally and log a warning 
message.
   
   - What did you see instead?
RocketMQ broker does not startup and does not explain the reason clearly.
   
   2. Please tell us about your environment:
   It's an ordinary CentOS 7.5 running Intel x86 chip on bare metal.
   
   3. Other information (e.g. detailed explanation, logs, related issues, 
suggestions how to fix, etc):
   
   This suggestion is to ignore file length is not same as mappedFileSize which 
is 1G by default. Log a warning and continue to start.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] francisoliverlee commented on issue #3257: Ignore irrelevant commit log files on startup

2021-08-12 Thread GitBox


francisoliverlee commented on issue #3257:
URL: https://github.com/apache/rocketmq/issues/3257#issuecomment-897773215


   checked it on 4.8.0 , you are right.  but data file not right is a FATAL 
ERROR, manually-check needed
   
![image](https://user-images.githubusercontent.com/5908412/129230632-bf714aae-5584-4c48-b4e7-742f61053865.png)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] francisoliverlee commented on issue #3255: add new allocate MessageQueue strategy

2021-08-12 Thread GitBox


francisoliverlee commented on issue #3255:
URL: https://github.com/apache/rocketmq/issues/3255#issuecomment-897784608


   nice work. and what circumstanceses it used for you ?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] coveralls commented on pull request #3256: [ISSUE #3257] Ignore irrelevant commit log files on startup

2021-08-12 Thread GitBox


coveralls commented on pull request #3256:
URL: https://github.com/apache/rocketmq/pull/3256#issuecomment-897788933


   
   [![Coverage 
Status](https://coveralls.io/builds/42102638/badge)](https://coveralls.io/builds/42102638)
   
   Coverage increased (+0.09%) to 54.01% when pulling 
**c43e99c3a6318a474431fc88707f7fa393b5546f on schnell18:master** into 
**a20f31bb3de242756542c552d7212a7a4ba4 on apache:master**.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] francisoliverlee commented on issue #3250: RocketMq is currently designed to have one or more consumequeues for each topic. Why not one or more consumequeues for each topic and tag

2021-08-12 Thread GitBox


francisoliverlee commented on issue #3250:
URL: https://github.com/apache/rocketmq/issues/3250#issuecomment-897795234


   interesting idea and how do you want to do this ?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-client-go] francisoliverlee commented on issue #711: [bug] The Unsubscribe() interface influence retrytopic

2021-08-12 Thread GitBox


francisoliverlee commented on issue #711:
URL: 
https://github.com/apache/rocketmq-client-go/issues/711#issuecomment-897860121


   ### as java client does in following files, client no need to Unsubscribe 
retry topic.
   
   > 
https://github.com/apache/rocketmq/blob/a20f31bb3de242756542c552d7212a7a4ba4/client/src/main/java/org/apache/rocketmq/client/consumer/DefaultMQPushConsumer.java#L792
   
   > 
https://github.com/apache/rocketmq/blob/a20f31bb3de242756542c552d7212a7a4ba4/client/src/main/java/org/apache/rocketmq/client/impl/consumer/DefaultMQPushConsumerImpl.java#L932
   
   ## as checked broker codes,  subscribtion for retry topic would never 
delete. broker update subscribtions on heartbeat:
   > 
https://github.com/apache/rocketmq/blob/a20f31bb3de242756542c552d7212a7a4ba4/broker/src/main/java/org/apache/rocketmq/broker/processor/ClientManageProcessor.java#L75


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-client-go] francisoliverlee opened a new pull request #713: [fix issue 711] The Unsubscribe() interface influence retrytopic

2021-08-12 Thread GitBox


francisoliverlee opened a new pull request #713:
URL: https://github.com/apache/rocketmq-client-go/pull/713


   ## What is the purpose of the change
   
   X
   
   ## Brief changelog
   
   XX
   
   ## Verifying this change
   
   
   
   Follow this checklist to help us incorporate your contribution quickly and 
easily. Notice, `it would be helpful if you could finish the following 5 
checklist(the last one is not necessary)before request the community to review 
your PR`.
   
   - [x] Make sure there is a [Github 
issue](https://github.com/apache/rocketmq/issues) filed for the change (usually 
before you start working on it). Trivial changes like typos do not require a 
Github issue. Your pull request should address just this issue, without pulling 
in other changes - one PR resolves one issue. 
   - [x] Format the pull request title like `[ISSUE #123] Fix UnknownException 
when host config not exist`. Each commit in the pull request should have a 
meaningful subject line and body.
   - [x] Write a pull request description that is detailed enough to understand 
what the pull request does, how, and why.
   - [x] Write necessary unit-test(over 80% coverage) to verify your logic 
correction, more mock a little better when a cross-module dependency exists.
   - [ ] If this contribution is large, please file an [Apache Individual 
Contributor License Agreement](http://www.apache.org/licenses/#clas).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-streams] duhenglucky merged pull request #21: Try to remove branch protect

2021-08-12 Thread GitBox


duhenglucky merged pull request #21:
URL: https://github.com/apache/rocketmq-streams/pull/21


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] 88888-IT commented on issue #3250: RocketMq is currently designed to have one or more consumequeues for each topic. Why not one or more consumequeues for each topic and tag

2021-08-12 Thread GitBox


8-IT commented on issue #3250:
URL: https://github.com/apache/rocketmq/issues/3250#issuecomment-898095819


   > interesting idea and how do you want to do this ?
   
   If the producer registers the topic and tag with the broker before sending 
messages, the broker can create four queues for the topic and tag. Is it 
possible to load balance a queue when a producer sends a message and then send 
the message to it? I don't know if that's feasible


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] wuyuan0717 commented on issue #3222: [question] Meaning of stats.log log output

2021-08-12 Thread GitBox


wuyuan0717 commented on issue #3222:
URL: https://github.com/apache/rocketmq/issues/3222#issuecomment-898099978


   > 
控制台上面的值是准确的,昨天一天的消息发送量。`BROKER_PUT_NUMS`你是用接口查询的嘛,如果是的话,那他是以当前时间为基础,过去24个小时的。是相对的
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] zhxjdwh commented on issue #3252: the rocketmq_client.log file spent too much cache/buffer memory

2021-08-12 Thread GitBox


zhxjdwh commented on issue #3252:
URL: https://github.com/apache/rocketmq/issues/3252#issuecomment-898128055


   @ni-ze  
   I have 400 jvm processes running in k8s, and each pod that uses Rocketmq 
consumes 2-6 times more memory than other pods (k8s counts buffer/cache into 
the used memory), and excessive memory consumption leads to frequent pods 
Killed by k8s.
   I can only adjust logLevel to ERROR, so that it consumes less memory
   
   
我有400个jvm进程运行在k8s,每个有用到rocketmq的pod都会比其他pod多消耗2-6倍的内存(k8s把buffer/cache算到已用内存里面),内存消耗过多导致pod频繁被k8s杀死。
   我现在只能把logLevel调整为ERROR,这样消耗内存就会少一点


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-spring] RongtongJin commented on issue #375: Transaction messages is not triggered

2021-08-12 Thread GitBox


RongtongJin commented on issue #375:
URL: https://github.com/apache/rocketmq-spring/issues/375#issuecomment-898131861


   What version?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-streams] RonzL opened a new pull request #22: add module db-operator、transport-minio

2021-08-12 Thread GitBox


RonzL opened a new pull request #22:
URL: https://github.com/apache/rocketmq-streams/pull/22


   Author:ronzl <1158402...@qq.com>


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-client-go] RongtongJin commented on a change in pull request #712: (fix): fix push_consumer#unsubscribe unsubscribe retry

2021-08-12 Thread GitBox


RongtongJin commented on a change in pull request #712:
URL: https://github.com/apache/rocketmq-client-go/pull/712#discussion_r688211430



##
File path: consumer/push_consumer.go
##
@@ -119,6 +119,9 @@ func NewPushConsumer(opts ...Option) (*pushConsumer, error) 
{
 
p.interceptor = primitive.ChainInterceptors(p.option.Interceptors...)
 
+   retryTopic := internal.GetRetryTopic(p.consumerGroup)
+   sub := buildSubscriptionData(retryTopic, MessageSelector{TAG, _SubAll})
+   p.subscriptionDataTable.Store(retryTopic, sub)

Review comment:
   Broadcast mode does not need to subscribe to retry topic




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] kingmax-yu commented on issue #3253: RockerMQ-4.7.1 storeerror.log reprot too many errorlog "WARN FlushConsumeQueueService - Offset not matched"

2021-08-12 Thread GitBox


kingmax-yu commented on issue #3253:
URL: https://github.com/apache/rocketmq/issues/3253#issuecomment-898157709


   WARN FlushConsumeQueueService - Offset not matched. Request offset: 
31800, firstOffset: 31200, lastOffset: 31800, mappedFileSize: 
600, mappedFiles count: 1
   2021-01-26 02:53:24 WARN ScheduleMessageTimerThread - Offset not matched. 
Request offset: 17928800, firstOffset: 1800, lastOffset: 2400, 
mappedFileSize: 600, mappedFiles count: 1
   2021-01-27 22:50:07 WARN AdminBrokerThread_11 - Offset not matched. Request 
offset: 0, firstOffset: 1200, lastOffset: 1800, mappedFileSize: 
600, mappedFiles count: 1
   我统计了一下,一共有三种
   
![1628823898(1)](https://user-images.githubusercontent.com/88824980/129299539-55910f44-894a-42fc-a4f9-9355af1d7800.png)
   也看了下源码,大致的意思就是 
当前的offset超出了最后一个mappedfile的最大偏移量,或者是比第一个mappedfile的最大偏移量还要小,也就是说如果比第一个文件的最大偏移量小的话,就说明已经在这个文件中被写过了,那么超出第二个文件的最大偏移量会怎么样呢?不知道我理解的对不对。
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-dashboard] vongosling opened a new issue #1: Are you using RocketMQ Dashboard?

2021-08-12 Thread GitBox


vongosling opened a new issue #1:
URL: https://github.com/apache/rocketmq-dashboard/issues/1


   ## Are you using RocketMQ Dashboard?
   
   The state-of-the-art Dashboard of Apache RoccketMQ provides excellent 
monitoring capability. Various graphs and statistics of events, performance and 
system information of clients and application is evidently made available to 
the user. If you are using RocketMQ Dashboard, first we would like to Thank 
You. Here, we sincerely invite you to take a minute to feedback on your usage 
scenario.
   
   ## What we expect from you
   Pls. submit a comment in this issue to include the following information:
   • your company, school, or organization
   • your country and city
   • your contact info, such as email, WeChat, and Twitter (optional).
   • usage scenario
   • expectations(optional)
   
   You can refer to the following sample answer for the format:
   ```
   * Organization: XX Company
   * Location: Seoul, South Korea
   * Contact: foo...@gmail.org
   * Version: v1.0.0
   * Status: production
   * Expectations(optional): Integrated with RocketMQ Operator, support 
   ```
   
   Thanks again for your participation!
   Apache RocketMQ Community


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq-streams] vongosling opened a new issue #23: Are you using RocketMQ Streams?

2021-08-12 Thread GitBox


vongosling opened a new issue #23:
URL: https://github.com/apache/rocketmq-streams/issues/23


   ## Are you using RocketMQ Streams?
   
   If you are using RocketMQ Streams, first we would like to Thank You. Here, 
we sincerely invite you to take a minute to feedback on your usage scenario.
   
   ## The purpose of this issue
   We are always interested in finding out who is using Streams, what attracted 
you to use it, how we can listen to your needs, and if you are interested, help 
promote your organization.
   
   ## What we would like from you
   Pls. submit a comment in this issue to include the following information:
   * your company, school, or organization
   * your country and city
   * your contact info, such as email, WeChat, and Twitter (optional).
   * usage scenario
   * expectations(optional)
   
   You can refer to the following sample answer for the format:
   ```
   * Organization: XX Company
   * Location: Seoul, South Korea
   * Contact: foo...@gmail.org
   * Version: v1.0.0
   * Status: production
   * Expectations(optional): Data ingest service
   ```
   
   Thanks again for your participation!
   Apache RocketMQ Community


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] Phoebus888 opened a new issue #3258: 生产关闭消息轨迹无效,

2021-08-12 Thread GitBox


Phoebus888 opened a new issue #3258:
URL: https://github.com/apache/rocketmq/issues/3258


   traceTopicEnable=false
   traceOn=false
   上面两个值我都改为false了,但是产生的主题RMQ_SYS_TRACE_TOPIC还是会产生大量消息造成集群压力负载


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] pirateskipper commented on issue #3258: 生产关闭消息轨迹无效,

2021-08-12 Thread GitBox


pirateskipper commented on issue #3258:
URL: https://github.com/apache/rocketmq/issues/3258#issuecomment-898199042


   Please describe the version and related environment.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] Phoebus888 commented on issue #3258: 生产关闭消息轨迹无效,

2021-08-12 Thread GitBox


Phoebus888 commented on issue #3258:
URL: https://github.com/apache/rocketmq/issues/3258#issuecomment-898200016


   版本为V4_7_1
   集群是双主双从   RMQ_SYS_TRACE_TOPIC路由到了一个节点上去,自动创建的
   
![image](https://user-images.githubusercontent.com/45359605/129308856-97bf3e90-ef45-41a2-8321-b485acbf041a.png)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] Phoebus888 commented on issue #3258: 生产关闭消息轨迹无效,

2021-08-12 Thread GitBox


Phoebus888 commented on issue #3258:
URL: https://github.com/apache/rocketmq/issues/3258#issuecomment-898200222


   > Please describe the version and related environment.
   
   现在消息轨迹产生的tps很高  影响了生产环境了


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [rocketmq] Phoebus888 commented on issue #3258: 生产关闭消息轨迹无效,

2021-08-12 Thread GitBox


Phoebus888 commented on issue #3258:
URL: https://github.com/apache/rocketmq/issues/3258#issuecomment-898201232


   ./mqadmin updateBrokerConfig -b 10.44.1.251:11011 -k traceOn -v false   
使用这个命令把所有节点都关闭了


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




  1   2   3   4   5   6   7   8   9   10   >