[
https://issues.apache.org/jira/browse/STORM-928?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14616780#comment-14616780
]
ASF GitHub Bot commented on STORM-928:
--------------------------------------
GitHub user dan-blanchard opened a pull request:
https://github.com/apache/storm/pull/624
[STORM-928] Add source->stream->fields mapping to multi-lang context
This pull request adds a `source->stream->fields` mapping to the topology
context sent to multi-lang components as part of the handshake.
It also clears up a few minor type issues in TopologyContext.java, and
fixes the Multi-Lang protocol doc to note that the additions I made in #525 are
in 0.10.0 and not 0.11.0.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/dan-blanchard/incubator-storm master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/storm/pull/624.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #624
----
commit a1912eafab7f4313c7f45f27748fff28cb37d8f0
Author: Dan Blanchard <[email protected]>
Date: 2015-07-06T20:45:29Z
Minor type cleanup things in storm.task.TopologyContext.java
commit 427cef54c74204a257bc97a3e8fb793854603aaf
Author: Dan Blanchard <[email protected]>
Date: 2015-07-07T14:25:58Z
Add source->stream->fields mapping to multi-lang handshake.
----
> Add sources->streams->fields map to Multi-Lang Handshake
> --------------------------------------------------------
>
> Key: STORM-928
> URL: https://issues.apache.org/jira/browse/STORM-928
> Project: Apache Storm
> Issue Type: Improvement
> Affects Versions: 0.10.0
> Reporter: Dan Blanchard
> Assignee: Dan Blanchard
> Labels: multilang
> Fix For: 0.10.0
>
>
> In my previous pull request for issue [STORM-789] I somehow omitted a key
> piece of information I wanted to add to the Multi-Lang handshake,
> "sources->streams->fields", which would allow multi-lang libraries to access
> Tuple values by field name instead of just by index. The pull request I am
> about to submit fixes this oversight. I hope that since this is directly
> related to something that was merged into the 0.10.x branch, that this fix
> can also make it in there.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
