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

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to