aahmed-se commented on pull request #10148:
URL: https://github.com/apache/pulsar/pull/10148#issuecomment-815149967


   Here is the updated pr https://github.com/apache/pulsar/pull/10158
   I have isolated all the unstable tests with extra group annotation, also 
there is template jos to run those tests on nightly cadence. This is temporary 
but suitable solution to unblock ci issues.
   
   ```When mixing class level @Test(group="...") definitions and method level 
@Test(group="...") definitions, it's necessary. My previous comment explains 
it. Perhaps you didn't have that type of use case?```
   
   I don't have that issue , we shouldn't be filtering things at the test 
method level, grouping should stay at the class level, mixing both is not a 
good idea.
   
   
   


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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to