[GitHub] spark pull request: Fix NPE for JsonProtocol

2014-07-23 Thread ScrapCodes
Github user ScrapCodes commented on the pull request: https://github.com/apache/spark/pull/1511#issuecomment-49902317 it would be good, if you could describe when do you get this NPE ? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark pull request: Fix NPE for JsonProtocol

2014-07-23 Thread mateiz
Github user mateiz commented on the pull request: https://github.com/apache/spark/pull/1511#issuecomment-49909067 Jenkins, test this please --- 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

[GitHub] spark pull request: Fix NPE for JsonProtocol

2014-07-23 Thread mateiz
Github user mateiz commented on the pull request: https://github.com/apache/spark/pull/1511#issuecomment-49909117 I think this can happen if you have a fetch-failed because the map output tracker deregistered a shuffle and didn't send you its info. --- If your project is set up for

[GitHub] spark pull request: Fix NPE for JsonProtocol

2014-07-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1511#issuecomment-49909560 QA tests have started for PR 1511. This patch merges cleanly. brView progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17045/consoleFull ---

[GitHub] spark pull request: Fix NPE for JsonProtocol

2014-07-23 Thread mateiz
Github user mateiz commented on the pull request: https://github.com/apache/spark/pull/1511#issuecomment-49928351 Actually one more thing is missing, @witgo: can you open a JIRA issue for this and include it in the pull request's title? After that I'll merge it. --- If your project

[GitHub] spark pull request: Fix NPE for JsonProtocol

2014-07-21 Thread witgo
GitHub user witgo opened a pull request: https://github.com/apache/spark/pull/1511 Fix NPE for JsonProtocol You can merge this pull request into a Git repository by running: $ git pull https://github.com/witgo/spark JsonProtocol Alternatively you can review and apply these

[GitHub] spark pull request: Fix NPE for JsonProtocol

2014-07-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1511#issuecomment-49614971 QA tests have started for PR 1511. This patch merges cleanly. brView progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16913/consoleFull ---

[GitHub] spark pull request: Fix NPE for JsonProtocol

2014-07-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1511#issuecomment-49629586 QA results for PR 1511:br- This patch PASSES unit tests.br- This patch merges cleanlybr- This patch adds no public classesbrbrFor more information see test