[phoenix] 03/03: PHOENIX-5658 IndexTool to verify index rows inline (addendum - removed unused the only-verify option)

2020-01-13 Thread kadir
This is an automated email from the ASF dual-hosted git repository.

kadir pushed a commit to branch 4.14-HBase-1.4
in repository https://gitbox.apache.org/repos/asf/phoenix.git

commit e2af500cdca11b02a59b69f42e500d52cb5f11b8
Author: Kadir 
AuthorDate: Sun Jan 12 19:18:33 2020 -0800

PHOENIX-5658 IndexTool to verify index rows inline (addendum - removed 
unused the only-verify option)
---
 .../src/main/java/org/apache/phoenix/mapreduce/index/IndexTool.java  | 5 -
 1 file changed, 5 deletions(-)

diff --git 
a/phoenix-core/src/main/java/org/apache/phoenix/mapreduce/index/IndexTool.java 
b/phoenix-core/src/main/java/org/apache/phoenix/mapreduce/index/IndexTool.java
index d07ed22..7d86ecf 100644
--- 
a/phoenix-core/src/main/java/org/apache/phoenix/mapreduce/index/IndexTool.java
+++ 
b/phoenix-core/src/main/java/org/apache/phoenix/mapreduce/index/IndexTool.java
@@ -207,10 +207,6 @@ public class IndexTool extends Configured implements Tool {
 "both before and after rebuilding row. If the verification 
is done before rebuilding rows and " +
 "the correct index rows are not rebuilt. Currently 
supported values are NONE, ONLY and AFTER ");
 
-private static final Option ONLY_VERIFY_OPTION = new Option("ov", 
"only-verify", false,
-"To verify every data table row has the corresponding index row 
with the correct content " +
-"(without building the index table). If the verify option is set 
then the only-verify option will be ignored");
-
 private static final double DEFAULT_SPLIT_SAMPLING_RATE = 10.0;
 
 private static final Option SPLIT_INDEX_OPTION =
@@ -252,7 +248,6 @@ public class IndexTool extends Configured implements Tool {
 options.addOption(PARTIAL_REBUILD_OPTION);
 options.addOption(DIRECT_API_OPTION);
 options.addOption(VERIFY_OPTION);
-options.addOption(ONLY_VERIFY_OPTION);
 options.addOption(RUN_FOREGROUND_OPTION);
 options.addOption(OUTPUT_PATH_OPTION);
 options.addOption(SNAPSHOT_OPTION);



[phoenix] 03/03: PHOENIX-5658 IndexTool to verify index rows inline (addendum - removed unused the only-verify option)

2020-01-13 Thread kadir
This is an automated email from the ASF dual-hosted git repository.

kadir pushed a commit to branch 4.14-HBase-1.3
in repository https://gitbox.apache.org/repos/asf/phoenix.git

commit 308013454c0d59cda21536382b7184f138e0d8c0
Author: Kadir 
AuthorDate: Sun Jan 12 19:18:33 2020 -0800

PHOENIX-5658 IndexTool to verify index rows inline (addendum - removed 
unused the only-verify option)
---
 .../src/main/java/org/apache/phoenix/mapreduce/index/IndexTool.java  | 5 -
 1 file changed, 5 deletions(-)

diff --git 
a/phoenix-core/src/main/java/org/apache/phoenix/mapreduce/index/IndexTool.java 
b/phoenix-core/src/main/java/org/apache/phoenix/mapreduce/index/IndexTool.java
index c2c32c9..8f5f3e3 100644
--- 
a/phoenix-core/src/main/java/org/apache/phoenix/mapreduce/index/IndexTool.java
+++ 
b/phoenix-core/src/main/java/org/apache/phoenix/mapreduce/index/IndexTool.java
@@ -207,10 +207,6 @@ public class IndexTool extends Configured implements Tool {
 "both before and after rebuilding row. If the verification 
is done before rebuilding rows and " +
 "the correct index rows are not rebuilt. Currently 
supported values are NONE, ONLY and AFTER ");
 
-private static final Option ONLY_VERIFY_OPTION = new Option("ov", 
"only-verify", false,
-"To verify every data table row has the corresponding index row 
with the correct content " +
-"(without building the index table). If the verify option is set 
then the only-verify option will be ignored");
-
 private static final double DEFAULT_SPLIT_SAMPLING_RATE = 10.0;
 
 private static final Option SPLIT_INDEX_OPTION =
@@ -252,7 +248,6 @@ public class IndexTool extends Configured implements Tool {
 options.addOption(PARTIAL_REBUILD_OPTION);
 options.addOption(DIRECT_API_OPTION);
 options.addOption(VERIFY_OPTION);
-options.addOption(ONLY_VERIFY_OPTION);
 options.addOption(RUN_FOREGROUND_OPTION);
 options.addOption(OUTPUT_PATH_OPTION);
 options.addOption(SNAPSHOT_OPTION);