[
https://issues.apache.org/jira/browse/STORM-1228?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15170374#comment-15170374
]
ASF GitHub Bot commented on STORM-1228:
---------------------------------------
GitHub user abellina opened a pull request:
https://github.com/apache/storm/pull/1160
STORM-1228: port fields_test to java
Fields::select can currently throw a null pointer exception. Apparently
this happens when _index.get("x") cannot find the field "x" returning null.
Since it's being used as an integer in tuple.get, the null value attempts but
can't unbox to integer.
Should it call Fields::fieldIndex, which will throw an
IllegalArgumentException if the field does not exist?
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/abellina/storm
STORM-1228_port_fields_test_to_java
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/storm/pull/1160.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 #1160
----
commit 251b4d26ba5aa993254be014647a4c14a0f36bc0
Author: Alessandro Bellina <[email protected]>
Date: 2016-02-27T04:42:59Z
STORM-1228: port fields_test to java
----
> port backtype.storm.fields-test to java
> ----------------------------------------
>
> Key: STORM-1228
> URL: https://issues.apache.org/jira/browse/STORM-1228
> Project: Apache Storm
> Issue Type: New Feature
> Components: storm-core
> Reporter: Robert Joseph Evans
> Assignee: Alessandro Bellina
> Labels: java-migration, jstorm-merger
>
> This is a test that should be simple to move to JUnit
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
