hgromer commented on code in PR #7288:
URL: https://github.com/apache/hbase/pull/7288#discussion_r2349074093


##########
hbase-client/src/main/java/org/apache/hadoop/hbase/client/Admin.java:
##########
@@ -2651,4 +2651,7 @@ List<LogEntry> getLogEntries(Set<ServerName> serverNames, 
String logType, Server
    * Get the list of cached files
    */
   List<String> getCachedFilesList(ServerName serverName) throws IOException;
+
+  @InterfaceAudience.Private
+  void restoreBackupSystemTable(String snapshotName) throws IOException;

Review Comment:
   I'm not sure HBCK is handled much differently than what I have now. We 
expose HBCK "publicly" via `Connection#getHBCK`, however, we limit that method 
call to `@InterfaceAudience.LimitedPrivate(HBaseInterfaceAudience.HBCK)`. Is 
there much difference between exposing a public IA.Private method that executes 
an actions vs a public IA.Private method that returns a class which executes 
actions?



-- 
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: issues-unsubscr...@hbase.apache.org

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

Reply via email to