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
HIVE-25250: Fix TestHS2ImpersonationWithRemoteMS.testImpersonation #2404
Conversation
47604c3
to
266c1ca
Compare
@kgyrtkirk Could you please review the PR? |
please run a flaky check on your changes to prove that it fixes the issue |
Thanks for pointing to flaky check pipeline. I ran and got the green build there http://ci.hive.apache.org/job/hive-flaky-check/269/ Please review and merge the PR. |
I'm a little bit amazed that changing the default |
@kgyrtkirk completely aligned with you. So I have reverted the change and fix the test by adding retires with in the test cases. Got green build in flaky test also |
thank you! |
@kgyrtkirk Could you please merge the PR? |
8b42cb6
to
c578003
Compare
…hish Sharma, reviewed by Zoltan Haindrich) Signed-off-by: Sankar Hariappan <sankarh@apache.org> Closes (apache#2404)
…hish Sharma, reviewed by Zoltan Haindrich) Signed-off-by: Sankar Hariappan <sankarh@apache.org> Closes (apache#2404) (cherry picked from commit dc8891e)
What changes were proposed in this pull request?
Fix flaky test TestHS2ImpersonationWithRemoteMS.testImpersonation. It was already fixed as part of HIVE-25093 but that is not the right way to do. Raising the PR again to create separate patch.
Why are the changes needed?
It is failing intermittently and putting unwanted pressure on hive-precommit setup.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Yes