Skip to content

Cleanup jmx.properties from Presto Native docker images #26787

@aditi-pandit

Description

@aditi-pandit

Presto Native doesn't support jmx. But some places in our code could be packaging container images with jmx.properties files like https://github.com/prestodb/presto/blob/master/docker/etc/catalog/jmx.properties

This results in a broken experience as the co-ordinator shows the jmx catalog, but all the queries fail with

presto> use current;
USE
presto:current> show tables;
                                                                           Table
-----------------------------------------------------------------------------------------------------------------------------------------------------------
 com.amazonaws.management:type=awssdkmetrics
 com.amazonaws.management:type=awssdkmetrics/1
 com.facebook.airlift.discovery.client:name=announcer
 com.facebook.airlift.discovery.client:name=serviceinventory
 com.facebook.airlift.discovery.store:name=dynamic,type=distributedstore
 com.facebook.airlift.discovery.store:name=dynamic,type=httpremotestore
 com.facebook.airlift.discovery.store:name=dynamic,type=replicator
 com.facebook.airlift.event.client:name=eventclient
 com.facebook.airlift.http.client:name=fordiscoveryclient,type=httpclient
 com.facebook.airlift.http.client:name=fordynamicstore,type=httpclient
...

presto:current> select * from "com.facebook.presto.hive.metastore.thrift:name=iceberg,type=thrifthivemetastore";
Query 20251211_030048_00049_9crpg failed: Error from native plan checker: it != protocols().end() Protocol for connector jmx not registered

Metadata

Metadata

Labels

Type

No type

Projects

Status

🆕 Unprioritized

Status

🆕 Unprioritized

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions