You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by ch...@apache.org on 2019/12/02 10:33:49 UTC
[phoenix] branch 4.x-HBase-1.3 updated: PHOENIX-5599 Make
LocalHBaseState consistent between master and 4.x
This is an automated email from the ASF dual-hosted git repository.
chenglei pushed a commit to branch 4.x-HBase-1.3
in repository https://gitbox.apache.org/repos/asf/phoenix.git
The following commit(s) were added to refs/heads/4.x-HBase-1.3 by this push:
new 1f85c13 PHOENIX-5599 Make LocalHBaseState consistent between master and 4.x
1f85c13 is described below
commit 1f85c13a0935a0705f69b0122aa75ab8d6a634f2
Author: chenglei <ch...@apache.org>
AuthorDate: Mon Dec 2 18:33:10 2019 +0800
PHOENIX-5599 Make LocalHBaseState consistent between master and 4.x
---
.../hbase/index/covered/data/LocalHBaseState.java | 31 +++++++++-------------
1 file changed, 13 insertions(+), 18 deletions(-)
diff --git a/phoenix-core/src/main/java/org/apache/phoenix/hbase/index/covered/data/LocalHBaseState.java b/phoenix-core/src/main/java/org/apache/phoenix/hbase/index/covered/data/LocalHBaseState.java
index 583e7f4..56e731a 100644
--- a/phoenix-core/src/main/java/org/apache/phoenix/hbase/index/covered/data/LocalHBaseState.java
+++ b/phoenix-core/src/main/java/org/apache/phoenix/hbase/index/covered/data/LocalHBaseState.java
@@ -22,9 +22,7 @@ import java.util.Collection;
import java.util.List;
import org.apache.hadoop.hbase.Cell;
-import org.apache.hadoop.hbase.KeyValue;
import org.apache.hadoop.hbase.client.Mutation;
-import org.apache.hadoop.hbase.client.Result;
import org.apache.phoenix.hbase.index.covered.update.ColumnReference;
/**
@@ -32,20 +30,17 @@ import org.apache.phoenix.hbase.index.covered.update.ColumnReference;
*/
public interface LocalHBaseState {
- /**
- * @param m mutation for which we should get the current table state
- * @param toCover all the columns the current row state needs to cover; hint the underlying lookup
- * to save getting all the columns for the row
- * @param ignoreNewerMutations ignore mutations newer than m when determining current state. Useful
- * when replaying mutation state for partial index rebuild where writes succeeded to the data
- * table, but not to the index table.
- * @return the full state of the given row. Includes all current versions (even if they are not
- * usually visible to the client (unless they are also doing a raw scan)). Never returns a
- * <tt>null</tt> {@link Result} - instead, when there is not data for the row, returns a
- * {@link Result} with no stored {@link KeyValue}s.
- * @throws IOException if there is an issue reading the row
- */
- public List<Cell> getCurrentRowState(Mutation m, Collection<? extends ColumnReference> toCover, boolean ignoreNewerMutations)
- throws IOException;
-
+ /**
+ * @param m mutation for which we should get the current table state
+ * @param toCover all the columns the current row state needs to cover; hint the underlying lookup
+ * to save getting all the columns for the row
+ * @param ignoreNewerMutations ignore mutations newer than m when determining current state. Useful
+ * when replaying mutation state for partial index rebuild where writes succeeded to the data
+ * table, but not to the index table.
+ * @return the full state of the given row. Includes all current versions (even if they are not
+ * usually visible to the client (unless they are also doing a raw scan)),may return null.
+ * @throws IOException if there is an issue reading the row
+ */
+ public List<Cell> getCurrentRowState(Mutation m, Collection<? extends ColumnReference> toCover, boolean ignoreNewerMutations)
+ throws IOException;
}
\ No newline at end of file