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-27488: HPL/SQL Quote literals not behaving as expected #4476
Conversation
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.
+1
Kudos, SonarCloud Quality Gate passed! |
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.
LGTM. Thanks for addressing the review comments. Some very minor stuff that I will fix during merge.
public class TestUtils { | ||
|
||
@Test | ||
public void testShouldRemoveEscapeCharactersFromTheGivenInput() { |
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.
We should include the name of the method that is tested. Something like:
testUnquoteStringRemovesOneQuoteWhenTwoConsecutive
… M reviewed by Attila Turoczy, Stamatis Zampetakis) Closes apache#4476
… M reviewed by Attila Turoczy, Stamatis Zampetakis) Closes apache#4476
What changes were proposed in this pull request?
Now including single quote(') in the output if the last two characters are single quotes(').
Why are the changes needed?
To fix the issue 'HIVE-27488: HPL/SQL Quote literals not behaving as expected'.
Does this PR introduce any user-facing change?
No
Is the change a dependency upgrade?
No
How was this patch tested?
Tests are added.