Skip to content

Use engine-provided MBeanServer in plugins #27640

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wendigo wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Use engine-provided MBeanServer in plugins #27640

wendigo wants to merge 1 commit into from

Conversation

@wendigo
Copy link
Contributor

@wendigo wendigo commented Dec 12, 2025 •
edited
Loading

This allows the engine in the future whether to expose or not to expose mbeans

Description

Additional context and related issues

Release notes

(x) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
( ) Release notes are required, with the following suggested text:

## Section
* Fix some things. ({issue}`issuenumber`)

@cla-bot cla-bot bot added the cla-signed label Dec 12, 2025
@github-actions github-actions bot added hudi Hudi connector iceberg Iceberg connector delta-lake Delta Lake connector hive Hive connector bigquery BigQuery connector mongodb MongoDB connector cassandra Cassandra connector elasticsearch Elasticsearch connector faker Faker connector google-sheets Google Sheets connector kafka Kafka connector loki Loki connector memory Memory connector opensearch OpenSearch connector pinot Pinot connector prometheus Prometheus connector redis Redis connector redshift Redshift connector lakehouse labels Dec 12, 2025
@wendigo wendigo force-pushed the serafin/mbean-contextualizes branch 3 times, most recently from 27de4e4 to eeaf684 Compare December 12, 2025 21:32
@wendigo wendigo requested a review from martint December 12, 2025 21:35
@wendigo wendigo force-pushed the serafin/mbean-contextualizes branch 2 times, most recently from 8a684f2 to de40ee0 Compare December 13, 2025 13:42
@github-actions github-actions bot added the postgresql PostgreSQL connector label Dec 13, 2025
@wendigo
Copy link
Contributor Author

wendigo commented Dec 17, 2025

Requires martint/jmxutils#66

@wendigo wendigo force-pushed the serafin/mbean-contextualizes branch from de40ee0 to 32c5100 Compare December 17, 2025 14:48
@github-actions
Copy link

github-actions bot commented Jan 7, 2026

This pull request has gone a while without any activity. Ask for help on #core-dev on Trino slack.

@github-actions github-actions bot added the stale label Jan 7, 2026
@wendigo
Use engine-provided MBeanServer in plugins
ba04767 
This allows the engine in the future whether to expose or not to expose mbeans
@wendigo wendigo force-pushed the serafin/mbean-contextualizes branch from 32c5100 to ba04767 Compare January 7, 2026 17:25
@github-actions github-actions bot removed the stale label Jan 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@martint martint Awaiting requested review from martint

Assignees

No one assigned

Labels

bigquery BigQuery connector cassandra Cassandra connector cla-signed delta-lake Delta Lake connector elasticsearch Elasticsearch connector faker Faker connector google-sheets Google Sheets connector hive Hive connector hudi Hudi connector iceberg Iceberg connector kafka Kafka connector lakehouse loki Loki connector memory Memory connector mongodb MongoDB connector opensearch OpenSearch connector pinot Pinot connector postgresql PostgreSQL connector prometheus Prometheus connector redis Redis connector redshift Redshift connector

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@wendigo