This is an automated email from the ASF dual-hosted git repository.

ndimiduk pushed a commit to branch branch-3
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-3 by this push:
     new 9d2cd41a491 HBASE-29179: Fall back to hbase.rpc.rows.warning.threshold 
if hbase.client.write.buffer.maxmutations is not set (#6769)
9d2cd41a491 is described below

commit 9d2cd41a49198d7d5f6bd770e3f0e0dda68ab2b2
Author: Charles Connell <[email protected]>
AuthorDate: Mon Mar 10 07:05:33 2025 -0500

    HBASE-29179: Fall back to hbase.rpc.rows.warning.threshold if 
hbase.client.write.buffer.maxmutations is not set (#6769)
    
    Signed-off-by: Nick Dimiduk <[email protected]>
---
 .../org/apache/hadoop/hbase/client/AsyncConnectionConfiguration.java  | 4 ++--
 hbase-common/src/main/resources/hbase-default.xml                     | 2 ++
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git 
a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncConnectionConfiguration.java
 
b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncConnectionConfiguration.java
index 05fe89ae237..1852895cdd1 100644
--- 
a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncConnectionConfiguration.java
+++ 
b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncConnectionConfiguration.java
@@ -204,8 +204,8 @@ class AsyncConnectionConfiguration {
       
TimeUnit.MICROSECONDS.toNanos(conf.getLong(HBASE_CLIENT_META_REPLICA_SCAN_TIMEOUT,
         HBASE_CLIENT_META_REPLICA_SCAN_TIMEOUT_DEFAULT));
     this.maxKeyValueSize = conf.getInt(MAX_KEYVALUE_SIZE_KEY, 
MAX_KEYVALUE_SIZE_DEFAULT);
-    this.bufferedMutatorMaxMutations =
-      conf.getInt(BUFFERED_MUTATOR_MAX_MUTATIONS_KEY, 
BUFFERED_MUTATOR_MAX_MUTATIONS_DEFAULT);
+    this.bufferedMutatorMaxMutations = 
conf.getInt(BUFFERED_MUTATOR_MAX_MUTATIONS_KEY,
+      conf.getInt(HConstants.BATCH_ROWS_THRESHOLD_NAME, 
BUFFERED_MUTATOR_MAX_MUTATIONS_DEFAULT));
   }
 
   long getMetaOperationTimeoutNs() {
diff --git a/hbase-common/src/main/resources/hbase-default.xml 
b/hbase-common/src/main/resources/hbase-default.xml
index 95da593082d..ccb1b28f870 100644
--- a/hbase-common/src/main/resources/hbase-default.xml
+++ b/hbase-common/src/main/resources/hbase-default.xml
@@ -1917,6 +1917,8 @@ possible configurations would overwhelm and obscure the 
important.
     <value>5000</value>
     <description>
       Number of rows in a batch operation above which a warning will be logged.
+      If hbase.client.write.buffer.maxmutations is not set, this will be used 
as
+      fallback for that setting.
     </description>
   </property>
   <property>

Reply via email to