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

Fixed NPE due to TNode processing before CNode #564

Merged
merged 1 commit into from Jan 7, 2021

Conversation

andsel
Copy link
Collaborator

@andsel andsel commented Jan 7, 2021

This PR added accessors methods to topic, in this way the TNode could throw exception if it's called. This let us fix #505 and #529, implementing the suggestion in #505 to move forward the TNode check.

Close #505
Close #529


How to test
usually the error appeared with a couple of run of:

sudo docker run --rm --network host inovex/mqtt-stresser -broker tcp://localhost:1883 -num-clients 40 -num-messages 150 -rampup-delay 1s -rampup-size 10 -global-timeout 180s -timeout 20s

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

Successfully merging this pull request may close these issues.

NullPointerException in CTrie.evaluate Subscription breaks during performance test
1 participant