You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@phoenix.apache.org by GitBox <gi...@apache.org> on 2020/05/19 19:46:06 UTC

[GitHub] [phoenix] gjacoby126 commented on a change in pull request #782: PHOENIX-5875 - Optional logging for IndexTool verification

gjacoby126 commented on a change in pull request #782:
URL: https://github.com/apache/phoenix/pull/782#discussion_r427556930



##########
File path: phoenix-core/src/main/java/org/apache/phoenix/mapreduce/index/IndexTool.java
##########
@@ -235,6 +272,11 @@ public static IndexVerifyType fromValue(byte[] value) {
     private static final Option END_TIME_OPTION = new Option("et", "endtime",
             true, "End time for indextool rebuild or verify");
 
+    private static final Option DISABLE_LOGGING_OPTION = new Option("dl",
+        "disable-logging", true
+        , "Disable logging of failed verification rows for BEFORE, " +

Review comment:
       @priyankporwal ONLY counts as BEFORE. 




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org