This is an automated email from the ASF dual-hosted git repository. vjasani pushed a commit to branch 5.2 in repository https://gitbox.apache.org/repos/asf/phoenix.git
The following commit(s) were added to refs/heads/5.2 by this push: new c05fdb1abe PHOENIX-7533 ADDENDUM to fix HBase 2.4 build c05fdb1abe is described below commit c05fdb1abea2da8136f9596220f375df98974c18 Author: Viraj Jasani <vjas...@apache.org> AuthorDate: Tue Apr 15 17:17:50 2025 -0700 PHOENIX-7533 ADDENDUM to fix HBase 2.4 build --- .../apache/phoenix/query/ConnectionQueryServicesImpl.java | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/phoenix-core-client/src/main/java/org/apache/phoenix/query/ConnectionQueryServicesImpl.java b/phoenix-core-client/src/main/java/org/apache/phoenix/query/ConnectionQueryServicesImpl.java index 81ec4c8e7a..9af457489f 100644 --- a/phoenix-core-client/src/main/java/org/apache/phoenix/query/ConnectionQueryServicesImpl.java +++ b/phoenix-core-client/src/main/java/org/apache/phoenix/query/ConnectionQueryServicesImpl.java @@ -22,7 +22,6 @@ import static org.apache.hadoop.hbase.client.ColumnFamilyDescriptorBuilder.MAX_V import static org.apache.hadoop.hbase.client.ColumnFamilyDescriptorBuilder.REPLICATION_SCOPE; import static org.apache.hadoop.hbase.client.ColumnFamilyDescriptorBuilder.TTL; import static org.apache.hadoop.hbase.client.MetricsConnection.CLIENT_SIDE_METRICS_ENABLED_KEY; -import static org.apache.hadoop.hbase.client.MetricsConnection.METRICS_SCOPE_KEY; import static org.apache.phoenix.coprocessorclient.MetaDataProtocol.MIN_SYSTEM_TABLE_TIMESTAMP; import static org.apache.phoenix.coprocessorclient.MetaDataProtocol.MIN_SYSTEM_TABLE_TIMESTAMP_4_15_0; import static org.apache.phoenix.coprocessorclient.MetaDataProtocol.MIN_SYSTEM_TABLE_TIMESTAMP_4_16_0; @@ -154,7 +153,6 @@ import org.apache.hadoop.hbase.client.Increment; import org.apache.hadoop.hbase.client.Mutation; import org.apache.hadoop.hbase.client.Put; import org.apache.hadoop.hbase.client.Result; -import org.apache.hadoop.hbase.client.RpcConnectionRegistry; import org.apache.hadoop.hbase.client.Table; import org.apache.hadoop.hbase.client.TableDescriptor; import org.apache.hadoop.hbase.client.TableDescriptorBuilder; @@ -219,6 +217,7 @@ import org.apache.phoenix.iterate.TableResultIterator.RenewLeaseStatus; import org.apache.phoenix.jdbc.ConnectionInfo; import org.apache.phoenix.jdbc.PhoenixConnection; import org.apache.phoenix.jdbc.PhoenixDatabaseMetaData; +import org.apache.phoenix.jdbc.RPCConnectionInfo; import org.apache.phoenix.log.ConnectionLimiter; import org.apache.phoenix.log.DefaultConnectionLimiter; import org.apache.phoenix.log.LoggingConnectionLimiter; @@ -455,8 +454,8 @@ public class ConnectionQueryServicesImpl extends DelegateQueryServices implement this.config.get(HConstants.CLIENT_ZOOKEEPER_CLIENT_PORT), HConstants.ZOOKEEPER_CLIENT_PORT, this.config.get(HConstants.ZOOKEEPER_CLIENT_PORT), - RpcConnectionRegistry.BOOTSTRAP_NODES, - this.config.get(RpcConnectionRegistry.BOOTSTRAP_NODES), + RPCConnectionInfo.BOOTSTRAP_NODES, + this.config.get(RPCConnectionInfo.BOOTSTRAP_NODES), HConstants.MASTER_ADDRS_KEY, this.config.get(HConstants.MASTER_ADDRS_KEY), ConnectionInfo.CLIENT_CONNECTION_REGISTRY_IMPL_CONF_KEY, this.config.get(ConnectionInfo.CLIENT_CONNECTION_REGISTRY_IMPL_CONF_KEY)); @@ -520,7 +519,10 @@ public class ConnectionQueryServicesImpl extends DelegateQueryServices implement } if (this.config.getBoolean(CLIENT_SIDE_METRICS_ENABLED_KEY, false)) { - this.config.set(METRICS_SCOPE_KEY, config.get(QUERY_SERVICES_NAME)); + // "hbase.client.metrics.scope" defined on + // org.apache.hadoop.hbase.client.MetricsConnection#METRICS_SCOPE_KEY + // however we cannot use the constant directly as long as we support HBase 2.4 profile. + this.config.set("hbase.client.metrics.scope", config.get(QUERY_SERVICES_NAME)); } if (!QueryUtil.isServerConnection(props)) {