/jolokia

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

Support for non-string keys when serializing TabularData #97

Closed
rhuss opened this Issue Apr 18, 2013 · 2 comments

Comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
  1.1.2
1 participant
@rhuss
@rhuss
Owner

rhuss commented Apr 18, 2013

Up to now, when accessing an MBean attribute of type TabularData with a path, then only string keys are supported for the path attributes.

See this dicussion . This is the spot in the source.

Since we already know how to convert a string to an OpenType this should be a big deal, however, at this place we unfortunately don't have access to this string-to-object converter.

For the short term (1.1.2), I'll add (and somewhat duplicate) conversion string-to-simple type in the TabularDataExtractor.

For the mid to long term (2.0) I'm doing quite some refactoring in order to improve modularity. Here it will be possible for the TabularDataExtractor to access the other converters, too and hence can convert any open type from a string.

rhuss added a commit that referenced this issue Apr 18, 2013

@rhuss
Quick fix for #97 to support more types than strings when using a path. 
This fix will be reconsidered for 2.0 where we hopefully will have the original
string-to-object converter in place within the TabularData extractor so that
the code doesn't need to be duplicated and more types will be supported.
Loading status checks…
e14e7be
@rhuss

This comment has been minimized.

Sign in to view
Owner

rhuss commented Apr 18, 2013

Quick fix added, I also just pushed out a 1.1.2-SNAPSHOT to labs.consol.de
@rhuss

This comment has been minimized.

Sign in to view
Owner

rhuss commented May 29, 2013

Fixed.

@rhuss rhuss closed this May 29, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment