Skip to content

MINOR: Fix Jmxtool to enforce wait option when Object is not yet available in MBean server#13995

Merged
omkreddy merged 1 commit intoapache:trunkfrom
omkreddy:jmxtool-fix
Jul 12, 2023
Merged

MINOR: Fix Jmxtool to enforce wait option when Object is not yet available in MBean server#13995
omkreddy merged 1 commit intoapache:trunkfrom
omkreddy:jmxtool-fix

Conversation

@omkreddy
Copy link
Contributor

@omkreddy omkreddy commented Jul 11, 2023

In JmxTool.scala, we will wait till all the object names are available from MBean server. But in the newer version, we only wait for subset of object names. Due to this, we may not enforce wait option and prematurely return the result if the objects are not yet registered in MBean sever.

Ran the system tests with the fix.

@omkreddy omkreddy changed the title MINOR: Fix Jmxtool to enforce wait option when MBean is not yet available in MBean server MINOR: Fix Jmxtool to enforce wait option when Object is not yet available in MBean server Jul 11, 2023
@omkreddy omkreddy marked this pull request as ready for review July 12, 2023 06:08
@omkreddy omkreddy requested review from fvaleri and mimaison July 12, 2023 06:08
Copy link
Contributor

@fvaleri fvaleri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @omkreddy, good catch. Thanks for fixing.

Copy link
Member

@showuon showuon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Nice catch!

@omkreddy
Copy link
Contributor Author

Test failures are not related. Merging to trunk and 3.5

@omkreddy omkreddy merged commit 4e85bc9 into apache:trunk Jul 12, 2023
omkreddy added a commit that referenced this pull request Jul 12, 2023
…le in MBean server (#13995)

In JmxTool.scala, we will wait till all the object names are available from MBean server. But in the newer version, we only wait for subset of object names. Due to this, we may not enforce wait option and prematurely return the result if the objects are not yet registered in MBean sever.

Reviewers: Luke Chen <showuon@gmail.com>, Federico Valeri <fvaleri@redhat.com>
Cerchie pushed a commit to Cerchie/kafka that referenced this pull request Jul 25, 2023
…le in MBean server (apache#13995)

In JmxTool.scala, we will wait till all the object names are available from MBean server. But in the newer version, we only wait for subset of object names. Due to this, we may not enforce wait option and prematurely return the result if the objects are not yet registered in MBean sever.

Reviewers: Luke Chen <showuon@gmail.com>, Federico Valeri <fvaleri@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants