[GitHub] spark pull request: [SPARK-2517] Removed some compiler type erasur...

2014-07-16 Thread rxin
GitHub user rxin opened a pull request: https://github.com/apache/spark/pull/1431 [SPARK-2517] Removed some compiler type erasure warnings. Also took the chance to rename some variables to avoid unintentional shadowing. You can merge this pull request into a Git repository by

[GitHub] spark pull request: [SPARK-2517] Removed some compiler type erasur...

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

[GitHub] spark pull request: [SPARK-2517] Removed some compiler type erasur...

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

[GitHub] spark pull request: [SPARK-2517] Removed some compiler type erasur...

2014-07-16 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/1431#issuecomment-49193038 How about we close this one and merge #1444? --- 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

[GitHub] spark pull request: [SPARK-2517] Removed some compiler type erasur...

2014-07-16 Thread rxin
Github user rxin closed the pull request at: https://github.com/apache/spark/pull/1431 --- 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

[GitHub] spark pull request: [SPARK-2517] Removed some compiler type erasur...

2014-07-16 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/1431#issuecomment-49202256 SGTM. --- 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