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 02:21:31 UTC

[GitHub] [pulsar] gaoran10 commented on a diff in pull request #17387: [fix][test] Fix `ClientDeduplicationFailureTest.testClientDeduplicationWithBkFailure`

gaoran10 commented on code in PR #17387:
URL: https://github.com/apache/pulsar/pull/17387#discussion_r961231092


##########
pulsar-broker/src/test/java/org/apache/pulsar/client/api/ClientDeduplicationFailureTest.java:
##########
@@ -60,7 +59,7 @@
 import org.testng.annotations.Test;
 
 @Slf4j
-@Test(groups = "quarantine")
+@Test(groups = "broker")

Review Comment:
   Maybe we can change the test group in another PR.
   
   @lhotari I think it's hard to judge whether the test is flaky or not, maybe the flaky problem was fixed in some PRs. Maybe we need to find a way to observe the failed frequency of the flaky tests.



-- 
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