[
https://issues.apache.org/jira/browse/STORM-1565?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15176230#comment-15176230
]
ASF GitHub Bot commented on STORM-1565:
---------------------------------------
Github user dan-blanchard commented on a diff in the pull request:
https://github.com/apache/storm/pull/1136#discussion_r54769764
--- Diff: conf/defaults.yaml ---
@@ -226,7 +226,7 @@ topology.eventlogger.executors: null
topology.tasks: null
# maximum amount of time a message has to complete before it's considered
failed
topology.message.timeout.secs: 30
-topology.multilang.serializer: "org.apache.storm.multilang.JsonSerializer"
+topology.multilang.serializer:
"org.apache.storm.multilang.MessagePackSerializer"
--- End diff --
While having the `MessagePackSeriailzer` built-in would be amazing,
changing it to the default serializer would break a lot of multi-lang
libraries, so I don't think that's a great idea. Let people opt in.
> Multi-Lang Performance Improvements
> -----------------------------------
>
> Key: STORM-1565
> URL: https://issues.apache.org/jira/browse/STORM-1565
> Project: Apache Storm
> Issue Type: Improvement
> Components: storm-core
> Reporter: Xin Wang
> Assignee: Xin Wang
>
> 1. add _org.apache.storm.multilang.MessagePackSerializer_
> 2. change default "topology.multilang.serializer" to _MessagePackSerializer_
> According to http://msgpack.org/ : It's like JSON, but fast and small.
> {quote}
> MessagePack is an efficient binary serialization format. It lets you exchange
> data among multiple languages like JSON. But it's faster and smaller. Small
> integers are encoded into a single byte, and typical short strings require
> only one extra byte in addition to the strings themselves.
> {quote}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
