GitHub user echauchot opened a pull request:

    https://github.com/apache/beam/pull/2659

    [BEAM-2029] null pointer exception while evaluating group by key in spark 
runner in streaming mode

    Be sure to do all of the following to help us incorporate your contribution
    quickly and easily:
    
     - [X] Make sure the PR title is formatted like:
       `[BEAM-<Jira issue #>] Description of pull request`
     - [ ] Make sure tests pass via `mvn clean verify`. (Even better, enable
           Travis-CI on your fork and ensure the whole test matrix passes).
     - [X] Replace `<Jira issue #>` in the title with the actual Jira issue
           number, if there is one.
     - [X] If this contribution is large, please file an Apache
           [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.pdf).
    
    ---
    R: @aviemzur 
    This PR, for now, just adds a test that raises a null pointer exception in 
spark runner so it cannot be merged as it is now.  I don't know Spark runner 
well enough for now to do the fix. But I suspect that the problem is due to the 
`withFanout()` support in spark runner. 
    
    @aviemzur , can you take a  look at the test and do the fix? 
    As we need the fix to have the build pass and merge this PR, what do you 
think about forking this PR and do the fix there, I will merge your PR and we 
will put the global PR (test + fix) to review and take care that your commits 
do not get squashed?


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/echauchot/beam 
BEAM-2029-NullPointerException_while_evaluating_GroupByKey_in_Spark_Runner_in_streaming_mode

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/beam/pull/2659.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 #2659
    
----
commit b68f0c5c2b507db7b66ee451208c0e19071ad7fd
Author: Etienne Chauchot <[email protected]>
Date:   2017-04-21T14:41:48Z

    [BEAM-2029] NullPointerException while evaluating GroupByKey in Spark 
Runner in streaming mode
    Add test to reproduce the NullPointerException

----


---
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