petersomogyi commented on code in PR #152:
URL: 
https://github.com/apache/hbase-operator-tools/pull/152#discussion_r2390927708


##########
hbase-hbck2/src/main/java/org/apache/hbase/HBCK2.java:
##########
@@ -118,7 +116,7 @@ public class HBCK2 extends Configured implements 
org.apache.hadoop.util.Tool {
         put(SET_TABLE_STATE, Arrays.asList("setTableStateInMeta"));
         put(BYPASS, Arrays.asList("bypassProcedure"));
         put(SCHEDULE_RECOVERIES,
-          Arrays.asList("scheduleServerCrashProcedure", 
"scheduleServerCrashProcedures"));
+          Arrays.asList("scheduleServerCrashProcedures"));

Review Comment:
   This removal doesn't mean HBCK would work with an older HBase, this would 
just throw the exception before calling the method. The change in this line is 
relevant: 
https://github.com/apache/hbase-operator-tools/pull/152/files#diff-2b2dc12521ea8fad10d0e53fa0ce78fc7dd3037a648a3e5c5547948e76511afdR596
 
   
   However, there are other incompatibilities with HBase 3.0 and resolving them 
would be difficult to keep it compatible with older versions. I started to 
collect them in HBASE-29638



-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to