[
https://issues.apache.org/jira/browse/HBASE-18921?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16189891#comment-16189891
]
Ted Yu commented on HBASE-18921:
--------------------------------
{code}
900 } else if (cellScannerIndex == this.cells.length) {
901 return false;
{code}
Within each if block, there is return statement. So the 'else' keyword can be
omitted.
> Result.current() throws ArrayIndexOutOfBoundsException after calling advance()
> ------------------------------------------------------------------------------
>
> Key: HBASE-18921
> URL: https://issues.apache.org/jira/browse/HBASE-18921
> Project: HBase
> Issue Type: Bug
> Reporter: Maytee Chinavanichkit
> Assignee: Maytee Chinavanichkit
> Priority: Minor
> Fix For: 2.0.0, 1.4.0, 1.3.2, 1.5.0, 1.2.7
>
> Attachments: master.patch
>
>
> On a Result object, if current() method is called after advance() returns
> false, this throws an ArrayIndexOutOfBoundsException. The expectation here is
> that it should return a null value.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
