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 2020/12/04 07:08:34 UTC

[GitHub] [pulsar] merlimat opened a new issue #8825: Test BatchMessageIndexAckTest.testDoNotRecycleAckSetMultipleTimes is failing

merlimat opened a new issue #8825:
URL: https://github.com/apache/pulsar/issues/8825


   There's a bug in master and tests are failing.
   
   The issue was introduced in #8161 and the test `BatchMessageIndexAckTest.testDoNotRecycleAckSetMultipleTimes` is failing as a result, although apparently is not being run from Github. 
   
   ```
   FAILED: testDoNotRecycleAckSetMultipleTimes
   java.lang.IllegalStateException: recycled multiple times
   	at io.netty.util.Recycler$Stack.pop(Recycler.java:567)
   	at io.netty.util.Recycler.get(Recycler.java:174)
   	at org.apache.pulsar.common.util.collections.BitSetRecyclable.create(BitSetRecyclable.java:1200)
   	at org.apache.pulsar.client.impl.PersistentAcknowledgmentsGroupingTracker.addBatchIndexAcknowledgment(PersistentAcknowledgmentsGroupingTracker.java:188)
   	at org.apache.pulsar.client.impl.ConsumerImpl.doAcknowledge(ConsumerImpl.java:582)
   	at org.apache.pulsar.client.impl.ConsumerBase.doAcknowledgeWithTxn(ConsumerBase.java:503)
   	at org.apache.pulsar.client.impl.ConsumerBase.acknowledgeCumulativeAsync(ConsumerBase.java:463)
   	at org.apache.pulsar.client.impl.ConsumerBase.acknowledgeCumulativeAsync(ConsumerBase.java:448)
   	at org.apache.pulsar.client.impl.ConsumerBase.acknowledgeCumulative(ConsumerBase.java:348)
   	at org.apache.pulsar.client.impl.ConsumerBase.acknowledgeCumulative(ConsumerBase.java:339)
   	at org.apache.pulsar.client.impl.BatchMessageIndexAckTest.testDoNotRecycleAckSetMultipleTimes(BatchMessageIndexAckTest.java:231)
   ```
   
   We should also understand why the PR validation jobs are not running all the tests and which ones are missing.
   
   @congbobo @codelipenghui Please take a look.


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [pulsar] merlimat commented on issue #8825: Test BatchMessageIndexAckTest.testDoNotRecycleAckSetMultipleTimes is failing

Posted by GitBox <gi...@apache.org>.
merlimat commented on issue #8825:
URL: https://github.com/apache/pulsar/issues/8825#issuecomment-738921169


   Thanks for fixing this quickly


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [pulsar] codelipenghui commented on issue #8825: Test BatchMessageIndexAckTest.testDoNotRecycleAckSetMultipleTimes is failing

Posted by GitBox <gi...@apache.org>.
codelipenghui commented on issue #8825:
URL: https://github.com/apache/pulsar/issues/8825#issuecomment-738615312


   @merlimat Thanks, we will take a look soon.


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [pulsar] codelipenghui commented on issue #8825: Test BatchMessageIndexAckTest.testDoNotRecycleAckSetMultipleTimes is failing

Posted by GitBox <gi...@apache.org>.
codelipenghui commented on issue #8825:
URL: https://github.com/apache/pulsar/issues/8825#issuecomment-739877871


   close this issue via #8829


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [pulsar] codelipenghui closed issue #8825: Test BatchMessageIndexAckTest.testDoNotRecycleAckSetMultipleTimes is failing

Posted by GitBox <gi...@apache.org>.
codelipenghui closed issue #8825:
URL: https://github.com/apache/pulsar/issues/8825


   


----------------------------------------------------------------
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:
users@infra.apache.org