java.jmx

Ability to read multiple attributes in one call

Details

  • Type: Enhancement Enhancement
  • Status: Resolved Resolved
  • Priority: Minor Minor
  • Resolution: Completed
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None
  • Patch:
    Code and Test
  • Approval:
    Accepted

Description

Currently, the only options for reading mbean attribute values are to read a single attribute or to read all attributes. It's useful to have a middle ground where a select list of attributes may be read.

The attached patch updates the raw-read function to conditionally accept a sequence of attribute names and return a map of attribute names to attribute values (the same format that the mbean function returns).

Activity

Hide
Nick Bailey added a comment -

Tyler, the attached patch doesn't apply to the HEAD of the master branch. Can you rebase and re-attach?

Show
Nick Bailey added a comment - Tyler, the attached patch doesn't apply to the HEAD of the master branch. Can you rebase and re-attach?
Hide
Nick Bailey added a comment -

Also, looking at the patch, the way it is currently implemented specifying multiple attributes will cause the read function to bypass the 'objects->data' transformation. We should address that as well.

Show
Nick Bailey added a comment - Also, looking at the patch, the way it is currently implemented specifying multiple attributes will cause the read function to bypass the 'objects->data' transformation. We should address that as well.
Hide
Nick Bailey added a comment -

actually disregard that last comment, read things wrong.

Show
Nick Bailey added a comment - actually disregard that last comment, read things wrong.
Hide
Nick Bailey added a comment -

An updated patch that applies to the current head.

Show
Nick Bailey added a comment - An updated patch that applies to the current head.
Nick Bailey made changes -
Field Original Value New Value
Attachment 0001-Add-ability-to-read-multiple-attributes-at-once.patch [ 10951 ]
Hide
Nick Bailey added a comment -

Committed

Show
Nick Bailey added a comment - Committed
Nick Bailey made changes -
Resolution Completed [ 1 ]
Approval Accepted [ 10005 ]
Assignee Nick Bailey [ nickmbailey ]
Status Open [ 1 ] Resolved [ 5 ]

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: