tkhurana commented on code in PR #2041:
URL: https://github.com/apache/phoenix/pull/2041#discussion_r1887279534
##########
phoenix-core-client/src/main/java/org/apache/phoenix/jdbc/PhoenixConnection.java:
##########
@@ -761,13 +761,22 @@ public PTable getTable(String name) throws SQLException {
}
public PTable getTable(PTableKey key) throws SQLException {
- PTable table;
- try {
- table = getTableRef(key).getTable();
- } catch (TableNotFoundException e) {
- table = getTableNoCache(key.getName());
+ PTableRef tableRef = getTableRefNoTNFE(key);
+ if (tableRef == null && key.getTenantId() != null) {
+ tableRef = getTableRefNoTNFE(new PTableKey(null, key.getName()));
Review Comment:
> For queries we use tenant connection to query a multi-tenant base table
Where is that mandated ?
You need to think in terms of the entity you are reading and writing to. If
the entity is a TENANT VIEW use a tenant connection otherwise use a global
connection.
--
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]
