Skip to content
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.

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

Cleanup monitorType JMX #611

Merged
merged 1 commit into from
Feb 6, 2025

Conversation

alaturqua
Copy link
Contributor

@alaturqua alaturqua commented Feb 6, 2025

Description

Addressed feedback from @ebyhr from the origin PR. #472

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:

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

@cla-bot cla-bot bot added the cla-signed label Feb 6, 2025
@alaturqua alaturqua mentioned this pull request Feb 6, 2025
@alaturqua alaturqua changed the title cleanup monitorType JMX Cleanup monitorType JMX Feb 6, 2025
@alaturqua alaturqua requested a review from willmostly February 6, 2025 07:18
@alaturqua alaturqua self-assigned this Feb 6, 2025
@ebyhr ebyhr requested review from ebyhr and removed request for willmostly February 6, 2025 07:20
@ebyhr ebyhr merged commit b16801a into trinodb:main Feb 6, 2025
2 checks passed
@github-actions github-actions bot added this to the 14 milestone Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants