search

Netflix / archaius

Library for configuration management API
Apache License 2.0
2.46k stars 485 forks source link

Add support for DynamicByteArrayProperty #47

Closed cfregly closed 11 years ago

cfregly commented 11 years ago

is this something that has been discussed and purposely not added?

also, we need to make sure this can be properly displayed in a JMX viewer - similar to the other properties.

cfregly commented 11 years ago

allen-

any thought to this? i started down this path first by extending PropertyWrapper, etc, but ran into a few road blocks.

is this something that has been previously discussed?

lemme know.

-chris

allenxwang commented 11 years ago

I do not recall this has not been discussed before.

It seems to me that we have to update DynamicProperty to add byte array support. The underlying configuration itself has to also know which type of data should be treated as byte array, instead of translating them to string.

Is there a real use case to have byte array as a type of property?

cfregly commented 11 years ago

we worked around this. closing.