[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 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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 changes as the patch at:

https://github.com/apache/spark/pull/1511.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 #1511


commit b187cfe8dd1a7ce047f3a2b0e5f43da0630eec83
Author: GuoQiang Li wi...@qq.com
Date:   2014-07-21T14:41:32Z

Fix NPE for JsonProtocol




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 
ouptut:brhttps://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16913/consoleFull


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---