HBASE-30238 Address bulkload replication review comments

Author: juke-mini666

hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/HFileReplicator.java

@@ -79,7 +79,9 @@ public class HFileReplicator implements Closeable {
   public static final int REPLICATION_BULKLOAD_COPY_HFILES_PERTHREAD_DEFAULT = 10;
   /**
    * Bandwidth limit in MB/s for copying HFiles from source cluster during bulkload replication. 0
-   * means no limit. Can be changed dynamically via configuration reload.
+   * means no limit. Can be changed dynamically via configuration reload. A low limit can make the
+   * sink-side bulkload copy exceed the replication RPC timeout, so configure the timeout
+   * accordingly.
    */
   public static final String REPLICATION_BULKLOAD_COPY_BANDWIDTH_MB_KEY =
     "hbase.replication.bulkload.copy.bandwidth.mb";

hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSink.java

@@ -472,7 +472,7 @@ private void addNewTableEntryInMap(
     bulkLoadHFileMap.put(tableName, newFamilyHFilePathsList);
   }
 
-  private static String buildBulkLoadKey(String replicationClusterId, BulkLoadDescriptor bld) {
+  private String buildBulkLoadKey(String replicationClusterId, BulkLoadDescriptor bld) {
     return replicationClusterId + "#" + Bytes.toString(bld.getEncodedRegionName().toByteArray())
       + "#" + bld.getBulkloadSeqNum();
   }