From 0ca2307b7730df0a535162bc76a93d2f7988c116 Mon Sep 17 00:00:00 2001 From: Duo Zhang Date: Wed, 26 Jan 2022 13:39:58 +0800 Subject: [PATCH] Revert "HBASE-26688 Threads shared EMPTY_RESULT may lead to unexpected client job down. (#4048)" This reverts commit 087bb1342a14b20b0785a33a563563bfc66fcce8. --- .../main/java/org/apache/hadoop/hbase/client/Result.java | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/Result.java b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/Result.java index 138432aada32..1ef1633d193e 100644 --- a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/Result.java +++ b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/Result.java @@ -913,19 +913,16 @@ public CellScanner cellScanner() { @Override public Cell current() { - if (isEmpty() + if (cells == null || cellScannerIndex == INITIAL_CELLSCANNER_INDEX - || cellScannerIndex >= cells.length) { + || cellScannerIndex >= cells.length) return null; - } return this.cells[cellScannerIndex]; } @Override public boolean advance() { - if (isEmpty()) { - return false; - } + if (cells == null) return false; cellScannerIndex++; if (cellScannerIndex < this.cells.length) { return true;