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
Set transaction type only if unset #3226
Conversation
Currently, changing the transaction type in an application has no effect, because it is changed here again. Separating transaction types is necessary if you want to separate main requests from internal RPC requests for example.
💚 CLA has been signed |
👋 @kelunik Thanks a lot for your contribution! It may take some time before we review a PR, so even if you don’t see activity for some time, it does not mean that we have forgotten about it. Every once in a while we go through a process of prioritization, after which we are focussing on the tasks that were planned for the upcoming milestone. The prioritization status is typically reflected through the PR labels. It could be pending triage, a candidate for a future milestone, or have a target milestone set to it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution!
It indeed does make sense to allow overriding the transaction type.
I've added a testcase proving your fix works as desired and added a changelog for you.
Could you please sign the CLA so that we can merge this PR?
Hey @JonasKunz, Thanks for the review and test! I have the CLA on my list, unfortunately it needs to go through legal before I can submit it. |
run elasticsearch-ci/docs |
run elasticsearch-ci/docs |
Thanks for your contribution @kelunik ! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
approved
@JonasKunz Thank you for the tests and handling with the CLA! |
What does this PR do?
This PR avoids overwriting a custom transaction type if set by the application.
Currently, changing the transaction type in an application has no effect, because it is changed by
ServletTransactionHelper
again.Separating transaction types is necessary if you want to separate main requests from internal RPC requests for example.
Checklist
Added an API method or config option? Document in which version this will be introducedI have made corresponding changes to the documentation