Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Intermittent test fail on BatchMessageTest.testSimpleBatchProducerConsumer #13

Closed
merlimat opened this issue Sep 13, 2016 · 4 comments
Closed

Comments

@merlimat
Copy link
Contributor

Output from Travis build:

com.yahoo.pulsar.client.admin.internal.BrokerStatsImpl$$EnhancerByMockitoWithCGLIB$$fb883741@6b353df8
Tests run: 361, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 831.053 sec <<< FAILURE! - in TestSuite
testSimpleBatchProducerConsumer(com.yahoo.pulsar.broker.service.BatchMessageTest)  Time elapsed: 0.306 sec  <<< FAILURE!
java.lang.AssertionError: expected [0] but found [4]
    at com.yahoo.pulsar.broker.service.BatchMessageTest.testSimpleBatchProducerConsumer(BatchMessageTest.java:293)

@merlimat merlimat added this to the 1.15 milestone Sep 13, 2016
@radekg
Copy link
Contributor

radekg commented Sep 24, 2016

FYI: I have not experienced this while working on #26

@radekg
Copy link
Contributor

radekg commented Sep 27, 2016

@merlimat I just tried the following pr https://github.com/radekg/pulsar/pull/5/files on this test, modified, changed the invocationCount

    @Test(invocationCount = 1000)
    public void testNonBatchCumulativeAckAfterBatchPublish() throws Exception {
      ...

and I do not have any failures locally. However, looking at the failure in another random place on the linked PR (direct travis URI: https://travis-ci.org/radekg/pulsar/builds/163260260), I think the intermittent errors are caused by using sync tests on an async code.

@radekg
Copy link
Contributor

radekg commented Sep 28, 2016

The next commit passes: https://travis-ci.org/radekg/pulsar/builds/163267361. Slightest blip on the main thread and some tests collapse.

@merlimat merlimat modified the milestones: 1.15, 1.16 Oct 17, 2016
@merlimat merlimat modified the milestones: 1.16, 1.17 Jan 23, 2017
@merlimat merlimat removed this from the 1.17 milestone Feb 23, 2017
@merlimat
Copy link
Contributor Author

Didn't see this failure in long time, closing for now

sijie referenced this issue in sijie/pulsar Mar 4, 2018
* Split functions module into api/runtime/api-examples

* Move `org.apache.pulsar.functions.api` package from runtime to api package
merlimat referenced this issue in merlimat/pulsar Sep 10, 2019
hangc0276 referenced this issue in hangc0276/pulsar May 26, 2021
currently, class KafkaServiceConfiguration extends ServiceConfiguration, but all fields in ServiceConfiguration is private, so it is not able to set these fields.
The main change of this PR is also getSuperclass().getDeclaredFields() when setting these fields.
eolivelli added a commit that referenced this issue Nov 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants