[GitHub] [pulsar] devinbost commented on issue #6224: Possible race condition: ReaderTest.testReadMessageWithBatchingWithMessageInclusive:148 expected [true] but found [false]

2020-02-14 Thread GitBox
devinbost commented on issue #6224: Possible race condition: ReaderTest.testReadMessageWithBatchingWithMessageInclusive:148 expected [true] but found [false] URL: https://github.com/apache/pulsar/issues/6224#issuecomment-586558853 I have a surefire log output with this issue here: [or

[GitHub] [pulsar] devinbost commented on issue #6224: Possible race condition: ReaderTest.testReadMessageWithBatchingWithMessageInclusive:148 expected [true] but found [false]

2020-02-04 Thread GitBox
devinbost commented on issue #6224: Possible race condition: ReaderTest.testReadMessageWithBatchingWithMessageInclusive:148 expected [true] but found [false] URL: https://github.com/apache/pulsar/issues/6224#issuecomment-582215418 In the failing test, `incomingMessages.size()` in `hasMoreM

[GitHub] [pulsar] devinbost commented on issue #6224: Possible race condition: ReaderTest.testReadMessageWithBatchingWithMessageInclusive:148 expected [true] but found [false]

2020-02-04 Thread GitBox
devinbost commented on issue #6224: Possible race condition: ReaderTest.testReadMessageWithBatchingWithMessageInclusive:148 expected [true] but found [false] URL: https://github.com/apache/pulsar/issues/6224#issuecomment-582214395 In a passing test, in: ``` private boolean hasMor

[GitHub] [pulsar] devinbost commented on issue #6224: Possible race condition: ReaderTest.testReadMessageWithBatchingWithMessageInclusive:148 expected [true] but found [false]

2020-02-04 Thread GitBox
devinbost commented on issue #6224: Possible race condition: ReaderTest.testReadMessageWithBatchingWithMessageInclusive:148 expected [true] but found [false] URL: https://github.com/apache/pulsar/issues/6224#issuecomment-582213865 So, the behavior must be different in `ConsumerImpl.hasMore

[GitHub] [pulsar] devinbost commented on issue #6224: Possible race condition: ReaderTest.testReadMessageWithBatchingWithMessageInclusive:148 expected [true] but found [false]

2020-02-04 Thread GitBox
devinbost commented on issue #6224: Possible race condition: ReaderTest.testReadMessageWithBatchingWithMessageInclusive:148 expected [true] but found [false] URL: https://github.com/apache/pulsar/issues/6224#issuecomment-582213571 The values of lastDequeuedMessage are the same between pass

[GitHub] [pulsar] devinbost commented on issue #6224: Possible race condition: ReaderTest.testReadMessageWithBatchingWithMessageInclusive:148 expected [true] but found [false]

2020-02-04 Thread GitBox
devinbost commented on issue #6224: Possible race condition: ReaderTest.testReadMessageWithBatchingWithMessageInclusive:148 expected [true] but found [false] URL: https://github.com/apache/pulsar/issues/6224#issuecomment-582213345 When the test passes, this call in ConsumerImpl.hasMessageA