tkhurana commented on a change in pull request #1328:
URL: https://github.com/apache/phoenix/pull/1328#discussion_r723883970
##########
File path: phoenix-core/src/main/java/org/apache/phoenix/util/ServerUtil.java
##########
@@ -328,7 +337,16 @@ public void shutdown() {
@Override
public HTableInterface getTable(ImmutableBytesPtr tablename,
ExecutorService pool)
throws IOException {
- return getConnection().getTable(tablename.copyBytesIfNecessary(),
pool);
+ ClusterConnection connection = null;
+ try {
+ connection = getConnection();
+ return connection.getTable(tablename.copyBytesIfNecessary(),
pool);
+ }catch (IllegalArgumentException e) {
Review comment:
@ankitjain64 can we refactor the two getTable functions to reduce
duplicate code. We can pass a null for the pool object and depending upon the
value use the corresponding connection.getTable() method.
##########
File path: phoenix-core/src/main/java/org/apache/phoenix/util/ServerUtil.java
##########
@@ -328,7 +337,16 @@ public void shutdown() {
@Override
public HTableInterface getTable(ImmutableBytesPtr tablename,
ExecutorService pool)
throws IOException {
- return getConnection().getTable(tablename.copyBytesIfNecessary(),
pool);
+ ClusterConnection connection = null;
+ try {
+ connection = getConnection();
+ return connection.getTable(tablename.copyBytesIfNecessary(),
pool);
+ }catch (IllegalArgumentException e) {
Review comment:
Won't something like this work
```
commonAPI(tablename, pool) {
if (pool == null)
connection.getTable(tablename);
else
connection.getTable(tablename, pool)
}
```
--
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]
