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

Support for non-string keys when serializing TabularData #97

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

Support for non-string keys when serializing TabularData #97

rhuss opened this issue Apr 18, 2013 · 2 comments
Milestone

Comments

@rhuss
Copy link
Member

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
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.
@rhuss
Copy link
Member Author

rhuss commented Apr 18, 2013

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

@rhuss
Copy link
Member Author

rhuss commented May 29, 2013

Fixed.

@rhuss rhuss closed this as completed May 29, 2013
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

No branches or pull requests

1 participant