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-26713: StringExpr ArrayIndexOutOfBoundsException with LIKE '%xxx%' #4999
Conversation
storage-api/src/test/org/apache/hadoop/hive/ql/exec/vector/expressions/TestStringExpr.java
Outdated
Show resolved
Hide resolved
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 +1. Pending a green test.
Would like to keep this PR open for a few days, so that let others who may be interested in this take a look.
storage-api/src/test/org/apache/hadoop/hive/ql/exec/vector/expressions/TestStringExpr.java
Outdated
Show resolved
Hide resolved
@ryukobayashi You can rebase this PR to get a green CI. |
Can you rebase your PR ? |
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
@zhangbutao Sorry for the late reply. The latest master is now green, so I merged it and this PR test also became green. |
Thanks. |
…%' (apache#4999)(Ryu Kobayashi, reviewed by Attila Turoczy, Butao Zhang)
…%' (apache#4999)(Ryu Kobayashi, reviewed by Attila Turoczy, Butao Zhang)
What changes were proposed in this pull request?
When LIKE(%xxx%) search is performed, if the character string contains control characters, overflow occurs.
Why are the changes needed?
ArrayIndexOutOfBoundsException will occur if the input data contains a control code, etc.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Added testcase as part of PR.