[
https://issues.apache.org/jira/browse/TEZ-1999?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14297873#comment-14297873
]
Hitesh Shah commented on TEZ-1999:
----------------------------------
Comments:
{code}
if (prevKey.getLength() <= 0) {
key.reset();
BufferUtils.copy(key, prevKey);
}
{code}
- Can key ever be empty or null?
- Also, when can prevKey length be less than 0?
- Might be good to have a more descriptive explanation instead of just a
jira id comment.
Any reason why the fix was not to remove the if check on line 492?
> IndexOutOfBoundsException during merge
> --------------------------------------
>
> Key: TEZ-1999
> URL: https://issues.apache.org/jira/browse/TEZ-1999
> Project: Apache Tez
> Issue Type: Bug
> Reporter: Rohini Palaniswamy
> Assignee: Rajesh Balamohan
> Attachments: TEZ-1999.1.patch
>
>
> This is most likely due to TEZ-1963
> b2 hits IndexOutOfBoundsException in
> public int compare(byte[] b1, int s1, int l1, byte[] b2, int s2, int l2) {
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
