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/27 10:42:11 UTC

[GitHub] [pulsar] codelipenghui commented on pull request #17746: [fix][flaky-test] Fix TransactionTest failure due to clean up after class

codelipenghui commented on PR #17746:
URL: https://github.com/apache/pulsar/pull/17746#issuecomment-1259317621

   @liangyepianzhou We should reduce the `beforeMethod` `afterMethod`, it will slow down the test, and waste the CI resources. The fix looks like moving some test from the `beforeClass` and `afterClass` to `beforeMethod` `afterMethod`. I think the right way is to make the test can work with `beforeClass` and `afterClass`, you can use `beforeMethod` and `afterMethod` to clean some metadata and data by using Pulsar Admin API.


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