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

ARTEMIS-2108 Potential StackOverflowError when load balancing disabled #128

Conversation

clebertsuconic
Copy link
Contributor

(cherry picked from commit d336a09)
(cherry picked from commit b6945b5)

downstream: ENTMQBR-1920

(cherry picked from commit d336a09)
(cherry picked from commit b6945b5)

downstream: ENTMQBR-1920
@andytaylor
Copy link
Contributor

@michalxo @rvais This doesnt change any semantics and will be covered by general regression testing so i would advise a qe ack

@rvais
Copy link

rvais commented Oct 15, 2018

@andytaylor we do not have this covered and it is a important customer issue. We have to cover it because now the customer issues are requirements for us. I can't acknowledge this PR. We need a task for it.

@rvais
Copy link

rvais commented Oct 15, 2018

adding ack so it can be merged and tested

andytaylor added a commit that referenced this pull request Oct 15, 2018
@andytaylor
Copy link
Contributor

merged

@andytaylor andytaylor closed this Oct 15, 2018
@clebertsuconic clebertsuconic deleted the ENTMQBR-1920 branch October 15, 2018 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants