You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by zh...@apache.org on 2020/03/27 02:29:17 UTC

[hbase] branch branch-2.3 updated: HBASE-24040 WALFactory.Providers.multiwal causes StackOverflowError (#1338)

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

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


The following commit(s) were added to refs/heads/branch-2.3 by this push:
     new a4c66b3  HBASE-24040 WALFactory.Providers.multiwal causes StackOverflowError (#1338)
a4c66b3 is described below

commit a4c66b3fa6f9761adb53114b33397d1d262c14fc
Author: WenFeiYi <we...@gmail.com>
AuthorDate: Fri Mar 27 10:04:52 2020 +0800

    HBASE-24040 WALFactory.Providers.multiwal causes StackOverflowError (#1338)
    
    Signed-off-by: Duo Zhang <zh...@apache.org>
---
 .../java/org/apache/hadoop/hbase/wal/RegionGroupingProvider.java    | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/wal/RegionGroupingProvider.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/wal/RegionGroupingProvider.java
index b3ee56c..2fd8288 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/wal/RegionGroupingProvider.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/wal/RegionGroupingProvider.java
@@ -118,7 +118,7 @@ public class RegionGroupingProvider implements WALProvider {
   public static final String REGION_GROUPING_STRATEGY = "hbase.wal.regiongrouping.strategy";
   public static final String DEFAULT_REGION_GROUPING_STRATEGY = Strategies.defaultStrategy.name();
 
-  /** delegate provider for WAL creation/roll/close */
+  /** delegate provider for WAL creation/roll/close, but not support multiwal */
   public static final String DELEGATE_PROVIDER = "hbase.wal.regiongrouping.delegate.provider";
   public static final String DEFAULT_DELEGATE_PROVIDER = WALFactory.Providers.defaultProvider
       .name();
@@ -159,6 +159,10 @@ public class RegionGroupingProvider implements WALProvider {
     }
     this.strategy = getStrategy(conf, REGION_GROUPING_STRATEGY, DEFAULT_REGION_GROUPING_STRATEGY);
     this.providerClass = factory.getProviderClass(DELEGATE_PROVIDER, DEFAULT_DELEGATE_PROVIDER);
+    if (providerClass.equals(this.getClass())) {
+      LOG.warn("delegate provider not support multiwal, falling back to defaultProvider.");
+      providerClass = factory.getDefaultProvider().clazz;
+    }
   }
 
   private WALProvider createProvider(String group) throws IOException {