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

NullPointerException in StoreAppender #1548

Open
nicktindall opened this issue Mar 18, 2024 · 5 comments
Open

NullPointerException in StoreAppender #1548

nicktindall opened this issue Mar 18, 2024 · 5 comments
Assignees

Comments

@nicktindall
Copy link
Contributor

nicktindall commented Mar 18, 2024

Failed test https://teamcity.chronicle.software/buildConfiguration/Chronicle_ChronicleQueueEnterprise_SnapshotJava11/941796?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&expandBuildTestsSection=true&hideProblemsFromDependencies=false&expandBuildProblemsSection=true&expandBuildChangesSection=true

This has failed in the same way numerous times. The wire field is marked as @Nullable but no null check is performed.

target/testAppendedSkipToEndMultiThreaded_7__DELTA_BINARY_wbm_None_rbm_None_named_false_encrypted_null_-1111-gufget2o2p
      Exception in thread "Thread-40" java.lang.NullPointerException
        at net.openhft.chronicle.queue.impl.single.StoreAppender$StoreAppenderContext.index(StoreAppender.java:1147)
        at net.openhft.chronicle.queue.impl.single.DeltaWireAppender.writingDocument(DeltaWireAppender.java:33)
        at net.openhft.chronicle.queue.impl.single.StoreAppender.writingDocument(StoreAppender.java:417)
        at net.openhft.chronicle.queue.impl.single.DeltaWireAppender.writingDocument(DeltaWireAppender.java:14)
        at net.openhft.chronicle.queue.impl.single.SingleChronicleQueueTest.writeTestDocument(SingleChronicleQueueTest.java:2596)
        at net.openhft.chronicle.queue.impl.single.SingleChronicleQueueTest.lambda$testAppendedSkipToEndMultiThreaded$90(SingleChronicleQueueTest.java:2376)
        at java.base/java.lang.Thread.run(Thread.java:829)
      Exception in thread "Thread-37" net.openhft.chronicle.core.io.ClosedIllegalStateException: Closed
        at net.openhft.chronicle.core.io.ManagedCloseable.throwExceptionIfClosed(ManagedCloseable.java:68)
        at net.openhft.chronicle.queue.impl.WireStorePool.acquire(WireStorePool.java:50)
        at net.openhft.chronicle.queue.impl.single.StoreAppender.setCycle2(StoreAppender.java:331)
        at net.openhft.chronicle.queue.impl.single.StoreAppender.setWireIfNull(StoreAppender.java:536)
        at net.openhft.chronicle.queue.impl.single.StoreAppender.prepareAndReturnWriteContext(StoreAppender.java:450)
        at net.openhft.chronicle.queue.impl.single.StoreAppender.writingDocument(StoreAppender.java:429)
        at net.openhft.chronicle.queue.impl.single.DeltaWireAppender.writingDocument(DeltaWireAppender.java:32)
        at net.openhft.chronicle.queue.impl.single.StoreAppender.writingDocument(StoreAppender.java:417)
        at net.openhft.chronicle.queue.impl.single.DeltaWireAppender.writingDocument(DeltaWireAppender.java:14)
        at net.openhft.chronicle.queue.impl.single.SingleChronicleQueueTest.writeTestDocument(SingleChronicleQueueTest.java:2596)
        at net.openhft.chronicle.queue.impl.single.SingleChronicleQueueTest.lambda$testAppendedSkipToEndMultiThreaded$90(SingleChronicleQueueTest.java:2376)
        at java.base/java.lang.Thread.run(Thread.java:829)
@tgd tgd added the review label Jun 3, 2024
@tgd tgd assigned tgd and yevgenp and unassigned tgd Jun 3, 2024
@yevgenp
Copy link
Contributor

yevgenp commented Jun 10, 2024

@tgd Is there any evidence that original issue still exists? I could not find any failed testAppendedSkipToEndMultiThreaded for the last month. Locally it also passes.

@tgd
Copy link
Contributor

tgd commented Jun 10, 2024

Hi @yevgenp - it is possible that the flaky history has been cleaned from TC as part of the scheduled or manual cleanup. Can you try "run until failure" in IDEA to see if you can trigger it?

@yevgenp
Copy link
Contributor

yevgenp commented Jun 10, 2024

I ran it locally for 10 minutes - no failures.

@tgd
Copy link
Contributor

tgd commented Jun 10, 2024

Thanks Yevgen - I also can't get it to fail on Linux - let's close it

@tgd tgd removed the review label Jun 10, 2024
@tgd
Copy link
Contributor

tgd commented Jun 10, 2024

Before closing, please can you check the callsite where the NPE occurs to see if there is any obvious reason why it may fail

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants