[GitHub] spark pull request: [SPARK-11226] [SQL] Empty line in json file sh...

2015-10-21 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/9211#issuecomment-150104148 Can you add a unit test for this? --- 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

[GitHub] spark pull request: [SPARK-11253][SQL] reset all accumulators in p...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9215#issuecomment-150106110 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-11253][SQL] reset all accumulators in p...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9215#issuecomment-150106101 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-10903] [SPARKR] R - Simplify SQLContext...

2015-10-21 Thread sun-rui
Github user sun-rui commented on a diff in the pull request: https://github.com/apache/spark/pull/9192#discussion_r42711903 --- Diff: R/pkg/R/SQLContext.R --- @@ -17,6 +17,34 @@ # SQLcontext.R: SQLContext-driven functions +#' Temporary function to reroute old

[GitHub] spark pull request: [SPARK-8334] [SQL] Binary logical plan should ...

2015-10-21 Thread navis
Github user navis closed the pull request at: https://github.com/apache/spark/pull/6790 --- 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-10971][SPARKR] RRunner should allow set...

2015-10-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9179#issuecomment-150108576 **[Test build #44130 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44130/consoleFull)** for PR 9179 at commit

[GitHub] spark pull request: [SPARK-10857] [SQL] Block SQL injection vulner...

2015-10-21 Thread rick-ibm
GitHub user rick-ibm opened a pull request: https://github.com/apache/spark/pull/9202 [SPARK-10857] [SQL] Block SQL injection vulnerabilities under DataFrameWriter.jdbc(). @marmbrus @rxin @joshrosen @srowen This patch attempts to address both

[GitHub] spark pull request: SPARK-1537 publisher-side code and tests

2015-10-21 Thread steveloughran
Github user steveloughran commented on a diff in the pull request: https://github.com/apache/spark/pull/8744#discussion_r42668737 --- Diff: yarn/history/src/main/scala/org/apache/spark/deploy/history/yarn/YarnTimelineUtils.scala --- @@ -0,0 +1,720 @@ +/* + * Licensed to

[GitHub] spark pull request: [SPARK-11243] [SQL] output UnsafeRow from colu...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9203#issuecomment-149998301 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-11243] [SQL] output UnsafeRow from colu...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9203#issuecomment-149998334 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-11243] [SQL] output UnsafeRow from colu...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9203#issuecomment-15251 Merged build finished. Test FAILed. --- 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-11243] [SQL] output UnsafeRow from colu...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9203#issuecomment-15256 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-11243] [SQL] output UnsafeRow from colu...

2015-10-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9203#issuecomment-15244 **[Test build #44083 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44083/consoleFull)** for PR 9203 at commit

[GitHub] spark pull request: [SPARK-10780] [ML] Set initialModel in KMeans ...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8972#issuecomment-150008005 Merged build finished. Test PASSed. --- 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-10780] [ML] Set initialModel in KMeans ...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8972#issuecomment-150008007 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10266][Documentation, ML] Fixed @Since ...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8729#issuecomment-150011604 Build triggered. --- 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: [SPARK-10666][SPARK-6880][CORE] Use properties...

2015-10-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6291#issuecomment-150012777 **[Test build #44078 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44078/consoleFull)** for PR 6291 at commit

[GitHub] spark pull request: [SPARK-10266][Documentation, ML] Fixed @Since ...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8729#issuecomment-150012763 Build finished. Test FAILed. --- 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

[GitHub] spark pull request: [SPARK-10666][SPARK-6880][CORE] Use properties...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6291#issuecomment-150012954 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-11244][SPARKR] sparkR.stop() should rem...

2015-10-21 Thread saurfang
GitHub user saurfang opened a pull request: https://github.com/apache/spark/pull/9205 [SPARK-11244][SPARKR] sparkR.stop() should remove SQLContext SparkR should remove `.sparkRSQLsc` and `.sparkRHivesc` when `sparkR.stop()` is called. Otherwise even when SparkContext is

[GitHub] spark pull request: [SPARK-10743][SQL] keep the name of expression...

2015-10-21 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/8859#issuecomment-150013714 Thanks, merging to master. --- 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

[GitHub] spark pull request: [SPARK-9622] [ML] DecisionTreeRegressor: provi...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8866#issuecomment-150013588 Merged build finished. Test PASSed. --- 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-9818][SQL][WIP]Revert SPARK-6136 to ena...

2015-10-21 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/8101#discussion_r42677306 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/jdbc/DockerHacks.scala --- @@ -0,0 +1,166 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-11116] [SQL] First Draft of Dataset API

2015-10-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9190#issuecomment-150014839 **[Test build #44087 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44087/consoleFull)** for PR 9190 at commit

[GitHub] spark pull request: [SPARK-10903] [SPARKR] R - Simplify SQLContext...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9192#issuecomment-149995521 Merged build finished. Test PASSed. --- 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-10903] [SPARKR] R - Simplify SQLContext...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9192#issuecomment-149995526 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-11242][SQL] In conf/spark-env.sh.templa...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9201#issuecomment-149996202 Can one of the admins verify this patch? --- 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-10641][WIP][SQL] Add Skewness and Kurto...

2015-10-21 Thread JihongMA
Github user JihongMA commented on a diff in the pull request: https://github.com/apache/spark/pull/9003#discussion_r42668167 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/functions.scala --- @@ -857,3 +857,329 @@ object

[GitHub] spark pull request: SPARK-1537 publisher-side code and tests

2015-10-21 Thread steveloughran
Github user steveloughran commented on a diff in the pull request: https://github.com/apache/spark/pull/8744#discussion_r42669119 --- Diff: yarn/history/src/main/scala/org/apache/spark/deploy/history/yarn/YarnTimelineUtils.scala --- @@ -0,0 +1,765 @@ +/* + * Licensed to

[GitHub] spark pull request: [SPARK-11243] [SQL] output UnsafeRow from colu...

2015-10-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9203#issuecomment-14732 **[Test build #44083 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44083/consoleFull)** for PR 9203 at commit

[GitHub] spark pull request: [SPARK-11236][CORE] Update Tachyon dependency ...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9204#issuecomment-150003057 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-11236][CORE] Update Tachyon dependency ...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9204#issuecomment-150003087 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-10266][Documentation, ML] Fixed @Since ...

2015-10-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8729#issuecomment-150012760 [Test build #44085 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44085/console) for PR 8729 at commit

[GitHub] spark pull request: [SPARK-10266][Documentation, ML] Fixed @Since ...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8729#issuecomment-150012765 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-9622] [ML] DecisionTreeRegressor: provi...

2015-10-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8866#issuecomment-150013422 **[Test build #44076 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44076/consoleFull)** for PR 8866 at commit

[GitHub] spark pull request: [SPARK-9622] [ML] DecisionTreeRegressor: provi...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8866#issuecomment-150013590 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10641][WIP][SQL] Add Skewness and Kurto...

2015-10-21 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/9003#discussion_r42677457 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/functions.scala --- @@ -857,3 +857,329 @@ object

[GitHub] spark pull request: [SPARK-11243] [SQL] output UnsafeRow from colu...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9203#issuecomment-150016227 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-11242][SQL] In conf/spark-env.sh.templa...

2015-10-21 Thread xguo27
GitHub user xguo27 opened a pull request: https://github.com/apache/spark/pull/9201 [SPARK-11242][SQL] In conf/spark-env.sh.template SPARK_DRIVER_MEMORY is documented incorrectly Minor fix on the comment You can merge this pull request into a Git repository by running: $ git

[GitHub] spark pull request: [SPARK-11098][Core]Add Outbox to cache the sen...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9197#issuecomment-149995302 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-11098][Core]Add Outbox to cache the sen...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9197#issuecomment-149995300 Merged build finished. Test PASSed. --- 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-10903] [SPARKR] R - Simplify SQLContext...

2015-10-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9192#issuecomment-149995360 **[Test build #44080 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44080/consoleFull)** for PR 9192 at commit

[GitHub] spark pull request: [SPARK-11163] Remove unnecessary addPendingTas...

2015-10-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9154#issuecomment-150009550 **[Test build #44079 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44079/consoleFull)** for PR 9154 at commit

[GitHub] spark pull request: [SPARK-11163] Remove unnecessary addPendingTas...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9154#issuecomment-150009623 Merged build finished. Test FAILed. --- 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-10979] [SparkR] Sparkrmerge: Add merge ...

2015-10-21 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/9012#issuecomment-150010329 Thanks @NarineK LGTM and I just had one minor inline comment. @sun-rui Any other comments ? --- If your project is set up for it, you can reply to this email

[GitHub] spark pull request: [SPARK-10266][Documentation, ML] Fixed @Since ...

2015-10-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8729#issuecomment-150012286 [Test build #44085 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44085/consoleFull) for PR 8729 at commit

[GitHub] spark pull request: [SPARK-11244][SPARKR] sparkR.stop() should rem...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9205#issuecomment-150013859 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-11116] [SQL] First Draft of Dataset API

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9190#issuecomment-150013834 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-11244][SPARKR] sparkR.stop() should rem...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9205#issuecomment-150013828 Merged build triggered. --- 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

[GitHub] spark pull request: Minor cleanup of ShuffleMapStage.outputLocs co...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9175#issuecomment-150013838 Merged build triggered. --- 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

[GitHub] spark pull request: Minor cleanup of ShuffleMapStage.outputLocs co...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9175#issuecomment-150013862 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-11116] [SQL] First Draft of Dataset API

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9190#issuecomment-150013863 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-10271][Pyspark][MLLib] Added @since tag...

2015-10-21 Thread noel-smith
Github user noel-smith commented on the pull request: https://github.com/apache/spark/pull/8627#issuecomment-149997391 Jenkins, retest 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

[GitHub] spark pull request: [SPARK-11243] [SQL] output UnsafeRow from colu...

2015-10-21 Thread davies
GitHub user davies opened a pull request: https://github.com/apache/spark/pull/9203 [SPARK-11243] [SQL] output UnsafeRow from columnar cache You can merge this pull request into a Git repository by running: $ git pull https://github.com/davies/spark unsafe_cache

[GitHub] spark pull request: [SPARK-10780] [ML] Set initialModel in KMeans ...

2015-10-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8972#issuecomment-149997721 **[Test build #44082 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44082/consoleFull)** for PR 8972 at commit

[GitHub] spark pull request: [SPARK-10641][WIP][SQL] Add Skewness and Kurto...

2015-10-21 Thread JihongMA
Github user JihongMA commented on the pull request: https://github.com/apache/spark/pull/9003#issuecomment-149997989 Seth, no need to implement the old interface, simply put a placeholder for resolving it is sufficient, which will go away when this code path is removed. here

[GitHub] spark pull request: [SPARK-11236][CORE] Update Tachyon dependency ...

2015-10-21 Thread calvinjia
GitHub user calvinjia opened a pull request: https://github.com/apache/spark/pull/9204 [SPARK-11236][CORE] Update Tachyon dependency from 0.7.1 -> 0.8.0. Upgrades the tachyon-client version to the latest release. No new dependencies are added and no spark facing APIs are

[GitHub] spark pull request: [SPARK-11236][CORE] Update Tachyon dependency ...

2015-10-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9204#issuecomment-150005057 **[Test build #44084 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44084/consoleFull)** for PR 9204 at commit

[GitHub] spark pull request: [SPARK-10780] [ML] Set initialModel in KMeans ...

2015-10-21 Thread yinxusen
Github user yinxusen commented on the pull request: https://github.com/apache/spark/pull/8972#issuecomment-150005925 @jayantshekhar Why not put the initialModel into ParamMap as we disussed in JIRA? --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark pull request: [SPARK-10780] [ML] Set initialModel in KMeans ...

2015-10-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8972#issuecomment-150007824 **[Test build #44082 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44082/consoleFull)** for PR 8972 at commit

[GitHub] spark pull request: [SPARK-10666][SPARK-6880][CORE] Use properties...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6291#issuecomment-150012949 Merged build finished. Test PASSed. --- 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: Minor cleanup of ShuffleMapStage.outputLocs co...

2015-10-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9175#issuecomment-150014390 **[Test build #44088 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44088/consoleFull)** for PR 9175 at commit

[GitHub] spark pull request: [SPARK-11244][SPARKR] sparkR.stop() should rem...

2015-10-21 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/9205#issuecomment-150015431 Thanks @saurfang -- Test is fine in the current location. Just curious what are the lint-r errors you saw ? --- If your project is set up for it, you can reply to

[GitHub] spark pull request: [SPARK-10708] Consolidate sort shuffle impleme...

2015-10-21 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/8829#issuecomment-150015488 Jenkins, retest 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

[GitHub] spark pull request: [SPARK-10857] [SQL] Block SQL injection vulner...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9202#issuecomment-149997285 Can one of the admins verify this patch? --- 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-10780] [ML] Set initialModel in KMeans ...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8972#issuecomment-149997293 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-10780] [ML] Set initialModel in KMeans ...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8972#issuecomment-149997313 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-11163] Remove unnecessary addPendingTas...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9154#issuecomment-150009626 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: SPARK-1537 publisher-side code and tests

2015-10-21 Thread steveloughran
Github user steveloughran commented on a diff in the pull request: https://github.com/apache/spark/pull/8744#discussion_r42674559 --- Diff: yarn/history/src/main/scala/org/apache/spark/deploy/history/yarn/YarnTimelineUtils.scala --- @@ -0,0 +1,720 @@ +/* + * Licensed to

[GitHub] spark pull request: [SPARK-10979] [SparkR] Sparkrmerge: Add merge ...

2015-10-21 Thread shivaram
Github user shivaram commented on a diff in the pull request: https://github.com/apache/spark/pull/9012#discussion_r42674571 --- Diff: R/pkg/R/DataFrame.R --- @@ -1457,15 +1457,142 @@ setMethod("join", dataFrame(sdf) }) -#' @rdname merge

[GitHub] spark pull request: [SPARK-10266][Documentation, ML] Fixed @Since ...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8729#issuecomment-150011645 Build started. --- 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

[GitHub] spark pull request: [SPARK-10979] [SparkR] Sparkrmerge: Add merge ...

2015-10-21 Thread falaki
Github user falaki commented on the pull request: https://github.com/apache/spark/pull/9012#issuecomment-150013102 +1 for more R-like merge signature. Thanks for doing it. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] spark pull request: [SPARK-11244][SPARKR] sparkR.stop() should rem...

2015-10-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9205#issuecomment-150015657 **[Test build #44086 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44086/consoleFull)** for PR 9205 at commit

[GitHub] spark pull request: [SPARK-10857] [SQL] Block SQL injection vulner...

2015-10-21 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/9202#issuecomment-150016295 ok to test --- 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

[GitHub] spark pull request: [SPARK-11243] [SQL] output UnsafeRow from colu...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9203#issuecomment-150019732 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-11243] [SQL] output UnsafeRow from colu...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9203#issuecomment-150019702 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-11244][SPARKR] sparkR.stop() should rem...

2015-10-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9205#issuecomment-150019771 **[Test build #44086 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44086/consoleFull)** for PR 9205 at commit

[GitHub] spark pull request: [SPARK-10857] [SQL] Block SQL injection vulner...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9202#issuecomment-150020934 Merged build finished. Test FAILed. --- 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-11243] [SQL] output UnsafeRow from colu...

2015-10-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9203#issuecomment-150020994 **[Test build #44092 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44092/consoleFull)** for PR 9203 at commit

[GitHub] spark pull request: [SPARK-10857] [SQL] Block SQL injection vulner...

2015-10-21 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/9202#issuecomment-150025997 Transient networking issues. 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

[GitHub] spark pull request: [SPARK-11208] [SQL] Filter out 'hive.metastore...

2015-10-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/9178 --- 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-10857] [SQL] Block SQL injection vulner...

2015-10-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9202#issuecomment-150027189 **[Test build #44094 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44094/consoleFull)** for PR 9202 at commit

[GitHub] spark pull request: [SPARK-11233][SQL] register cosh in function r...

2015-10-21 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/9199#issuecomment-150027004 Thanks - I've merged this. --- 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: [SPARK-8654] [SQL] Analysis exception when usi...

2015-10-21 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/9036#issuecomment-150027072 Thanks! I'll try and fix the conflicts while merging to master. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark pull request: [SPARK-11243] [SQL] output UnsafeRow from colu...

2015-10-21 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/9203#discussion_r42684787 --- Diff: sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/codegen/UnsafeRowWriter.java --- @@ -95,41 +99,75 @@ public void

[GitHub] spark pull request: [SPARK-10857] [SQL] Block SQL injection vulner...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9202#issuecomment-150027762 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10266][Documentation, ML] Fixed @Since ...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8729#issuecomment-150030288 Merged build finished. Test FAILed. --- 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-11243] [SQL] output UnsafeRow from colu...

2015-10-21 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/9203#discussion_r42686848 --- Diff: sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeRow.java --- @@ -618,6 +618,25 @@ public void writeTo(ByteBuffer buffer) {

[GitHub] spark pull request: [SPARK-11243] [SQL] output UnsafeRow from colu...

2015-10-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9203#issuecomment-150016965 **[Test build #44089 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44089/consoleFull)** for PR 9203 at commit

[GitHub] spark pull request: [SPARK-10708] Consolidate sort shuffle impleme...

2015-10-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8829#issuecomment-150018124 **[Test build #44090 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44090/consoleFull)** for PR 8829 at commit

[GitHub] spark pull request: [SPARK-10277][MLlib][PySpark] Add @since annot...

2015-10-21 Thread noel-smith
Github user noel-smith commented on the pull request: https://github.com/apache/spark/pull/8684#issuecomment-150018838 I think we should keep the comments - but just tweak them so that they're valid [reStructuredText](http://sphinx-doc.org/rest.html). In this case all that

[GitHub] spark pull request: [SPARK-9740] [SPARK-9592] [SPARK-9210] [SQL] C...

2015-10-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8113 --- 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-5929] Pyspark: Register a pip requireme...

2015-10-21 Thread buckhx
Github user buckhx commented on the pull request: https://github.com/apache/spark/pull/4897#issuecomment-150020186 @davies removed the requirements file from the context constructor. I looked into add --py-requirements to spark-submit, but it looked a bit more in depth than I was

[GitHub] spark pull request: [SPARK-11210][SPARKR][WIP] Add window function...

2015-10-21 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/spark/pull/9196#issuecomment-150021663 This is merging 2 PR/JIRA? --- 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

[GitHub] spark pull request: [SPARK-11163] Remove unnecessary addPendingTas...

2015-10-21 Thread kayousterhout
Github user kayousterhout commented on the pull request: https://github.com/apache/spark/pull/9154#issuecomment-150024132 Jenkins, retest 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

[GitHub] spark pull request: [SPARK-10271][Pyspark][MLLib] Added @since tag...

2015-10-21 Thread noel-smith
Github user noel-smith commented on the pull request: https://github.com/apache/spark/pull/8627#issuecomment-150025192 @yu-iskw Thanks for testing! @mengxr I've rebased now - the changes were very minor. Should be good to merge once the tests complete. --- If your project is set

[GitHub] spark pull request: [SPARK-11208] [SQL] Filter out 'hive.metastore...

2015-10-21 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/9178#issuecomment-150025412 Thanks, merging to master. --- 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

[GitHub] spark pull request: [SPARK-11233][SQL] register cosh in function r...

2015-10-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/9199 --- 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-10266][Documentation, ML] Fixed @Since ...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8729#issuecomment-150030293 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-11243] [SQL] output UnsafeRow from colu...

2015-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9203#issuecomment-150016246 Merged build started. --- 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

<    1   2   3   4   5   6   7   8   >