0001-Add-ability-to-read-multiple-attributes-at-once.patch
read-attributes-001.diff
[JMX-4] Ability to read multiple attributes in one call Created: 06/Feb/12 Updated: 23/Feb/12 Resolved: 23/Feb/12 |
|
| Status: | Resolved |
| Project: | java.jmx |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Enhancement | Priority: | Minor |
| Reporter: | Tyler Hobbs | Assignee: | Nick Bailey |
| Resolution: | Completed | Votes: | 0 |
| Labels: | None | ||
| Attachments: |
0001-Add-ability-to-read-multiple-attributes-at-once.patch
read-attributes-001.diff
|
| 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). |
| Comments |
| Comment by Nick Bailey [ 22/Feb/12 10:51 AM ] |
|
Tyler, the attached patch doesn't apply to the HEAD of the master branch. Can you rebase and re-attach? |
| Comment by Nick Bailey [ 23/Feb/12 2:34 PM ] |
|
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. |
| Comment by Nick Bailey [ 23/Feb/12 2:36 PM ] |
|
actually disregard that last comment, read things wrong. |
| Comment by Nick Bailey [ 23/Feb/12 7:22 PM ] |
|
An updated patch that applies to the current head. |
| Comment by Nick Bailey [ 23/Feb/12 7:23 PM ] |
|
Committed |