James Xu created STORM-170:
------------------------------
Summary: serialize all storm zk state as EDN
Key: STORM-170
URL: https://issues.apache.org/jira/browse/STORM-170
Project: Apache Storm (Incubating)
Issue Type: Improvement
Reporter: James Xu
Priority: Minor
https://github.com/nathanmarz/storm/issues/525
this will make it possible to add new entries without breaking backwards
compatibility.
I implemented push away from java serialization in issue #419 and pr #497. It
seems straightforward to change the serialization in this pr to EDN. Is this
what everyone would like me to do?
----------
hausdorff: I want to make sure I know specifically what you want, since I'm a
new contributor to the project.
Currently the pr mentioned above transitions zk serialization clojure s-exps,
represented as UTF-8 strings.
This issue proposes changing the zk serialization to EDN so we can "add new
entries without breaking backwards compatibility." I don't know EDN that well,
but the spec says it's a superset of Clojure, so will what I've written already
work? It's unclear to me whether the answer is yes, because I'm not actually
sure what you mean by adding "new entries."
----------
jasonjckn: clojure data is sufficient to meet "add new entries without breaking
backwards compatibility."
I suggest you wait for Nathan's feedback before doing anymore work, he's busy
doing a start-up though.
--
This message was sent by Atlassian JIRA
(v6.1.4#6159)
