[
https://issues.apache.org/jira/browse/STORM-170?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Rick Kellogg updated STORM-170:
-------------------------------
Component/s: storm-core
> serialize all storm zk state as EDN
> -----------------------------------
>
> Key: STORM-170
> URL: https://issues.apache.org/jira/browse/STORM-170
> Project: Apache Storm
> Issue Type: Improvement
> Components: storm-core
> 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.3.4#6332)
