Skip to content

Commit 7230d78

Browse files
committed
revert table client manager
1 parent 4fc32d4 commit 7230d78

File tree

1 file changed

+14
-20
lines changed

1 file changed

+14
-20
lines changed

src/main/java/com/alipay/oceanbase/hbase/util/ObTableClientManager.java

Lines changed: 14 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@
2222
import com.google.common.base.Objects;
2323
import org.apache.hadoop.classification.InterfaceAudience;
2424

25-
2625
import java.io.IOException;
2726
import java.util.Map;
2827
import java.util.Properties;
@@ -37,7 +36,7 @@
3736
public class ObTableClientManager {
3837

3938
public static final ConcurrentHashMap<ObTableClientKey, ReentrantLock> OB_TABLE_CLIENT_LOCK = new ConcurrentHashMap<ObTableClientKey, ReentrantLock>();
40-
public static final Map<ObTableClientKey, ObTableClient> OB_TABLE_CLIENT_INSTANCE = new ConcurrentHashMap<>();
39+
public static final Map<ObTableClientKey, ObTableClient> OB_TABLE_CLIENT_INSTANCE = new ConcurrentHashMap<ObTableClientKey, ObTableClient>();
4140

4241
public static ObTableClient getOrCreateObTableClient(OHConnectionConfiguration connectionConfig)
4342
throws IllegalArgumentException,
@@ -58,23 +57,18 @@ public static ObTableClient getOrCreateObTableClient(OHConnectionConfiguration c
5857
} else {
5958
checkArgument(isNotBlank(connectionConfig.getParamUrl()), HBASE_OCEANBASE_PARAM_URL
6059
+ " is blank");
61-
obTableClientKey = generateObTableClientKey(connectionConfig);
62-
}
63-
return getOrCreateObTableClient(obTableClientKey, connectionConfig.getRpcConnectTimeout());
64-
}
65-
66-
public static ObTableClientKey generateObTableClientKey(OHConnectionConfiguration connectionConfig) {
67-
ObTableClientKey obTableClientKey = new ObTableClientKey();
68-
String paramUrl = connectionConfig.getParamUrl();
69-
if (!paramUrl.contains("database")) {
70-
paramUrl += "&database=default";
71-
}
72-
obTableClientKey.setParamUrl(paramUrl);
73-
obTableClientKey.setSysUserName(connectionConfig.getSysUsername());
74-
if (connectionConfig.getSysPassword() == null) {
75-
obTableClientKey.setSysPassword(Constants.EMPTY_STRING);
76-
} else {
77-
obTableClientKey.setSysPassword(connectionConfig.getSysPassword());
60+
obTableClientKey = new ObTableClientKey();
61+
String paramUrl = connectionConfig.getParamUrl();
62+
if (!paramUrl.contains("database")) {
63+
paramUrl += "&database=default";
64+
}
65+
obTableClientKey.setParamUrl(paramUrl);
66+
obTableClientKey.setSysUserName(connectionConfig.getSysUsername());
67+
if (connectionConfig.getSysPassword() == null) {
68+
obTableClientKey.setSysPassword(Constants.EMPTY_STRING);
69+
} else {
70+
obTableClientKey.setSysPassword(connectionConfig.getSysPassword());
71+
}
7872
}
7973
checkArgument(isNotBlank(connectionConfig.getFullUsername()),
8074
HBASE_OCEANBASE_FULL_USER_NAME + " is blank");
@@ -90,7 +84,7 @@ public static ObTableClientKey generateObTableClientKey(OHConnectionConfiguratio
9084
obTableClientKey.getProperties().put(property.getKey(), property.getValue());
9185
}
9286

93-
return obTableClientKey;
87+
return getOrCreateObTableClient(obTableClientKey, connectionConfig.getRpcConnectTimeout());
9488
}
9589

9690
public static ObTableClient getOrCreateObTableClient(ObTableClientKey obTableClientKey,

0 commit comments

Comments
 (0)