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


##########
hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/BackupSystemTable.java:
##########
@@ -420,7 +421,8 @@ public void registerBulkLoad(TableName tableName, byte[] 
region,
    * @param rows the row keys of the entries to be deleted
    */
   public void deleteBulkLoadedRows(List<byte[]> rows) throws IOException {
-    try (BufferedMutator bufferedMutator = 
connection.getBufferedMutator(bulkLoadTableName)) {
+    try (BufferedMutator bufferedMutator = connection
+      .getBufferedMutator(new 
BufferedMutatorParams(bulkLoadTableName).setMaxMutations(1000))) {

Review Comment:
   Would it be possible to set this to the config value 
`HConstants.BATCH_ROWS_THRESHOLD_NAME`, seeing as that's the cluster limit for 
batches?
   
   The configuration value must then be supplied by the operator launching the 
backup. 
   



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