[GitHub] incubator-rocketmq issue #39: [ROCKETMQ-45]Delete hanged consume queue files
Github user lizhanhui commented on the issue: https://github.com/apache/incubator-rocketmq/pull/39 > We'd better not merge the PR if the opinion fails to unite Agree. But this is a minor change to fix obvious corner case mis-handling. Proposing review points are carefully considered and mostly accepted. Anyway, Let's wait for three review OK next time before merging. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] incubator-rocketmq issue #39: [ROCKETMQ-45]Delete hanged consume queue files
Github user vongosling commented on the issue: https://github.com/apache/incubator-rocketmq/pull/39 Usually, we need 3 committers to review PR. We'd better not merge the PR if the opinion fails to unite --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] incubator-rocketmq issue #39: [ROCKETMQ-45]Delete hanged consume queue files
Github user coveralls commented on the issue: https://github.com/apache/incubator-rocketmq/pull/39 [![Coverage Status](https://coveralls.io/builds/9784819/badge)](https://coveralls.io/builds/9784819) Coverage increased (+0.1%) to 25.814% when pulling **57111a02721ac8d70de3bb6210db4cf13f4714a9 on lizhanhui:ROCKETMQ-45** into **b29c318cdde225ef3a33a73e939e49e087766a28 on apache:master**. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] incubator-rocketmq issue #39: [ROCKETMQ-45]Delete hanged consume queue files
Github user coveralls commented on the issue: https://github.com/apache/incubator-rocketmq/pull/39 [![Coverage Status](https://coveralls.io/builds/9784819/badge)](https://coveralls.io/builds/9784819) Coverage increased (+0.1%) to 25.814% when pulling **57111a02721ac8d70de3bb6210db4cf13f4714a9 on lizhanhui:ROCKETMQ-45** into **b29c318cdde225ef3a33a73e939e49e087766a28 on apache:master**. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] incubator-rocketmq issue #39: [ROCKETMQ-45]Delete hanged consume queue files
Github user coveralls commented on the issue: https://github.com/apache/incubator-rocketmq/pull/39 [![Coverage Status](https://coveralls.io/builds/9784819/badge)](https://coveralls.io/builds/9784819) Coverage increased (+0.1%) to 25.814% when pulling **57111a02721ac8d70de3bb6210db4cf13f4714a9 on lizhanhui:ROCKETMQ-45** into **b29c318cdde225ef3a33a73e939e49e087766a28 on apache:master**. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] incubator-rocketmq issue #39: [ROCKETMQ-45]Delete hanged consume queue files
Github user coveralls commented on the issue: https://github.com/apache/incubator-rocketmq/pull/39 [![Coverage Status](https://coveralls.io/builds/9784804/badge)](https://coveralls.io/builds/9784804) Coverage increased (+0.1%) to 25.854% when pulling **b8e53c990e0b467c2a2b766137f7b32f3b5b6f04 on lizhanhui:ROCKETMQ-45** into **b29c318cdde225ef3a33a73e939e49e087766a28 on apache:master**. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] incubator-rocketmq issue #39: [ROCKETMQ-45]Delete hanged consume queue files
Github user coveralls commented on the issue: https://github.com/apache/incubator-rocketmq/pull/39 [![Coverage Status](https://coveralls.io/builds/9784804/badge)](https://coveralls.io/builds/9784804) Coverage increased (+0.1%) to 25.854% when pulling **b8e53c990e0b467c2a2b766137f7b32f3b5b6f04 on lizhanhui:ROCKETMQ-45** into **b29c318cdde225ef3a33a73e939e49e087766a28 on apache:master**. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] incubator-rocketmq issue #39: [ROCKETMQ-45]Delete hanged consume queue files
Github user coveralls commented on the issue: https://github.com/apache/incubator-rocketmq/pull/39 [![Coverage Status](https://coveralls.io/builds/9784804/badge)](https://coveralls.io/builds/9784804) Coverage increased (+0.1%) to 25.854% when pulling **b8e53c990e0b467c2a2b766137f7b32f3b5b6f04 on lizhanhui:ROCKETMQ-45** into **b29c318cdde225ef3a33a73e939e49e087766a28 on apache:master**. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] incubator-rocketmq issue #39: [ROCKETMQ-45]Delete hanged consume queue files
Github user zhouxinyu commented on the issue: https://github.com/apache/incubator-rocketmq/pull/39 @lizhanhui Also please add some unit tests -:).. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] incubator-rocketmq issue #39: [ROCKETMQ-45]Delete hanged consume queue files
Github user zhouxinyu commented on the issue: https://github.com/apache/incubator-rocketmq/pull/39 I checked ConsumeQueue#correctMinOffset, and you are right this method won't work as expected in this situation. Please @vintagewang @vongosling help review this PR. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] incubator-rocketmq issue #39: [ROCKETMQ-45]Delete hanged consume queue files
Github user lizhanhui commented on the issue: https://github.com/apache/incubator-rocketmq/pull/39 > what's the appearance/effect of this BUG? Reported bug description is consume queue files cannot be automatically deleted and took up large portion of disk. > IMO, it's ok when ConsumeQueue-MappedFile hold failed, this file will be released by the last holder, while the empty mapped file will retain in MappedFileQueue, but the min consume queue offset will be corrected. ConsumeQueue#correctMinOffset will not work as expected if consume queue file hanged. Double check the source code to figure out. > It seems that the only problem is the hanged consume queue only can be deleted when restart the broker. Not only this one. As correcting min offset is also affected, consumer client probably experiences long term of "commit log being deleted." before consuming progress catches up. There should be additional potential issues. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] incubator-rocketmq issue #39: [ROCKETMQ-45]Delete hanged consume queue files
Github user zhouxinyu commented on the issue: https://github.com/apache/incubator-rocketmq/pull/39 Hi, @lizhanhui , what's the appearance/effect of this BUG? Let's start from the next round. IMO, it's ok when ConsumeQueue-MappedFile hold failed, this file will be released by the last holder, while the empty mapped file will retain in MappedFileQueue, but the min consume queue offset will be corrected. It seems that the only problem is the hanged consume queue only can be deleted when restart the broker. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---