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-2023 Avoiding boolean on every message for 1x and tests #2281

Merged
merged 1 commit into from Aug 30, 2018

Conversation

clebertsuconic
Copy link
Contributor

No description provided.

@clebertsuconic clebertsuconic force-pushed the compatible-changes branch 3 times, most recently from 9eee8c1 to 02c1ed8 Compare August 29, 2018 20:51
@michaelandrepearce
Copy link
Contributor

michaelandrepearce commented Aug 30, 2018

Can we remove the need for core session to know about a jms 1x fix with this? Or is that already gone in the last change you did. I cant remember

@clebertsuconic
Copy link
Contributor Author

@michaelandrepearce it's not there any longer. it's just Session. Just using extension instead of boolean. that's all.

@clebertsuconic
Copy link
Contributor Author

For some reason compatibility-tests are broken. I didn't break compatibility tests.. but there's something I need to fix.

I planning this to be the last PR before releasing. (and cherry-pick)

@asfgit asfgit merged commit 93cffed into apache:master Aug 30, 2018
asfgit pushed a commit that referenced this pull request Aug 30, 2018
@clebertsuconic clebertsuconic deleted the compatible-changes branch July 30, 2019 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants