You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/09/02 03:40:05 UTC

[GitHub] [pulsar] poorbarcode opened a new pull request, #17418: [fix][flaky-test]DispatcherBlockConsumerTest.testBrokerSubscriptionRecovery

poorbarcode opened a new pull request, #17418:
URL: https://github.com/apache/pulsar/pull/17418

   ### Motivation
   
   test example: https://github.com/apache/pulsar/runs/8128077384?check_suite_focus=true
   
   #### Why flaky
   
   Consumers use subscription type `Shared` which is not guaranteed to be sequential but uses the index to comparison
   
   ### Modifications
   
   compare using the content of the unack messages
   
   ### Documentation
   
   
   - [ ] `doc-required` 
   
     
   - [x] `doc-not-needed` 
   
     
   - [ ] `doc` 
   
   
   - [ ] `doc-complete`
   


-- 
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: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] poorbarcode commented on pull request #17418: [fix][flaky-test]DispatcherBlockConsumerTest.testBrokerSubscriptionRecovery

Posted by GitBox <gi...@apache.org>.
poorbarcode commented on PR #17418:
URL: https://github.com/apache/pulsar/pull/17418#issuecomment-1235066927

   the problem still exists: https://github.com/apache/pulsar/runs/8149264299?check_suite_focus=true


-- 
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: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] poorbarcode closed pull request #17418: [fix][flaky-test]DispatcherBlockConsumerTest.testBrokerSubscriptionRecovery

Posted by GitBox <gi...@apache.org>.
poorbarcode closed pull request #17418: [fix][flaky-test]DispatcherBlockConsumerTest.testBrokerSubscriptionRecovery
URL: https://github.com/apache/pulsar/pull/17418


-- 
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: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] poorbarcode commented on pull request #17418: [fix][flaky-test]DispatcherBlockConsumerTest.testBrokerSubscriptionRecovery

Posted by GitBox <gi...@apache.org>.
poorbarcode commented on PR #17418:
URL: https://github.com/apache/pulsar/pull/17418#issuecomment-1235069002

   try expand wait time of receive message


-- 
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: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] poorbarcode closed pull request #17418: [fix][flaky-test]DispatcherBlockConsumerTest.testBrokerSubscriptionRecovery

Posted by GitBox <gi...@apache.org>.
poorbarcode closed pull request #17418: [fix][flaky-test]DispatcherBlockConsumerTest.testBrokerSubscriptionRecovery
URL: https://github.com/apache/pulsar/pull/17418


-- 
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: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] tisonkun commented on pull request #17418: [fix][flaky-test] PersistentStreamingDispatcherBlockConsumerTest.testBrokerSubscriptionRecovery

Posted by GitBox <gi...@apache.org>.
tisonkun commented on PR #17418:
URL: https://github.com/apache/pulsar/pull/17418#issuecomment-1235989028

   > Try to expand the wait time to receive messages
   
   Since we have a great number of tests, this approach can hardly work. Because if every test tries to wait for a long time, the resource is consumed more eagerly and more tests become unstable due to they don't get enough time to run.


-- 
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: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] poorbarcode commented on pull request #17418: [fix][flaky-test]DispatcherBlockConsumerTest.testBrokerSubscriptionRecovery

Posted by GitBox <gi...@apache.org>.
poorbarcode commented on PR #17418:
URL: https://github.com/apache/pulsar/pull/17418#issuecomment-1235447773

   @mattisonchao will fix the problem of `streaming dispatch`


-- 
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: commits-unsubscribe@pulsar.apache.org

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