[GitHub] spark pull request: [SPARK-10170] [SQL] Add DB2 JDBC dialect suppo...

2015-08-27 Thread lresende
Github user lresende commented on the pull request: https://github.com/apache/spark/pull/8393#issuecomment-135583856 LGTM --- 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

[GitHub] spark pull request: SPARK-9545, SPARK-9547: Use Maven in PRB if ti...

2015-08-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7878#issuecomment-135584151 [Test build #41711 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41711/consoleFull) for PR 7878 at commit [`cf58c49`](https://gith

[GitHub] spark pull request: [SPARK-10321] sizeInBytes in HadoopFsRelation

2015-08-27 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/8490#issuecomment-135584661 LGTM, merging to 1.4, 1.5, and 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 d

[GitHub] spark pull request: [SPARK-10170] [SQL] Add DB2 JDBC dialect suppo...

2015-08-27 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/8393#issuecomment-135584724 BTW - are there any way we can create a lightweight docker container with some version of DB2 for testing the JDBC connectivity to DB2? (both from a licensing pov and techn

[GitHub] spark pull request: [SPARK-10321] sizeInBytes in HadoopFsRelation

2015-08-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8490 --- 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 enab

[GitHub] spark pull request: [SPARK-10323] [SQL] fix nullability of In/InSe...

2015-08-27 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/8492#issuecomment-135588372 cc @yhuai --- 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 enab

[GitHub] spark pull request: [SPARK-10323] [SQL] fix nullability of In/InSe...

2015-08-27 Thread davies
GitHub user davies opened a pull request: https://github.com/apache/spark/pull/8492 [SPARK-10323] [SQL] fix nullability of In/InSet/ArrayContain After this PR, In/InSet/ArrayContain will return null if value is null, instead of false. They also will return null even if there is a nu

[GitHub] spark pull request: [SPARK-9905] Adds LinearRegressionSummary user...

2015-08-27 Thread feynmanliang
GitHub user feynmanliang opened a pull request: https://github.com/apache/spark/pull/8491 [SPARK-9905] Adds LinearRegressionSummary user guide * Adds user guide for LinearRegressionSummary * Fixes unresolved issues in #8197 You can merge this pull request into a Git repository

[GitHub] spark pull request: [SPARK-9905][ML][Doc] Adds LinearRegressionSum...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8491#issuecomment-135588711 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 h

[GitHub] spark pull request: [SPARK-10323] [SQL] fix nullability of In/InSe...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8492#issuecomment-135588705 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 h

[GitHub] spark pull request: [SPARK-10323] [SQL] fix nullability of In/InSe...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8492#issuecomment-135588718 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 have

[GitHub] spark pull request: [SPARK-9905][ML][Doc] Adds LinearRegressionSum...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8491#issuecomment-135588731 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 have

[GitHub] spark pull request: [SPARK-10323] [SQL] fix nullability of In/InSe...

2015-08-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8492#issuecomment-135589035 [Test build #41712 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41712/consoleFull) for PR 8492 at commit [`c3c65f8`](https://gith

[GitHub] spark pull request: [SPARK-9905][ML][Doc] Adds LinearRegressionSum...

2015-08-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8491#issuecomment-135589491 [Test build #41713 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41713/consoleFull) for PR 8491 at commit [`4e5aaeb`](https://gith

[GitHub] spark pull request: [SPARK-10065][SQL] Avoid triple copying of var...

2015-08-27 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/8484#issuecomment-135591299 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 have this fea

[GitHub] spark pull request: [SPARK-9905][ML][Doc] Adds LinearRegressionSum...

2015-08-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8491#issuecomment-135591430 [Test build #41713 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41713/console) for PR 8491 at commit [`4e5aaeb`](https://github.

[GitHub] spark pull request: [SPARK-10065][SQL] Avoid triple copying of var...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8484#issuecomment-135591401 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 h

[GitHub] spark pull request: [SPARK-10065][SQL] Avoid triple copying of var...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8484#issuecomment-135591410 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 have

[GitHub] spark pull request: [SPARK-9905][ML][Doc] Adds LinearRegressionSum...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8491#issuecomment-135591494 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 project

[GitHub] spark pull request: [SPARK-9905][ML][Doc] Adds LinearRegressionSum...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8491#issuecomment-135591495 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41

[GitHub] spark pull request: [SPARK-10323] [SQL] fix nullability of In/InSe...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8492#issuecomment-135591740 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41

[GitHub] spark pull request: [SPARK-10323] [SQL] fix nullability of In/InSe...

2015-08-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8492#issuecomment-135591715 [Test build #41712 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41712/console) for PR 8492 at commit [`c3c65f8`](https://github.

[GitHub] spark pull request: [SPARK-10323] [SQL] fix nullability of In/InSe...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8492#issuecomment-135591739 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 project

[GitHub] spark pull request: [SPARK-10065][SQL] Avoid triple copying of var...

2015-08-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8484#issuecomment-135592006 [Test build #41714 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41714/consoleFull) for PR 8484 at commit [`35371fb`](https://gith

[GitHub] spark pull request: [SPARK-4980] [MLlib] Add decay factors to stre...

2015-08-27 Thread rotationsymmetry
Github user rotationsymmetry commented on a diff in the pull request: https://github.com/apache/spark/pull/8022#discussion_r38162690 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/regression/StreamingLinearRegressionWithSGD.scala --- @@ -47,6 +52,7 @@ class StreamingLinearR

[GitHub] spark pull request: [SPARK-4980] [MLlib] Add decay factors to stre...

2015-08-27 Thread rotationsymmetry
Github user rotationsymmetry commented on a diff in the pull request: https://github.com/apache/spark/pull/8022#discussion_r38163035 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/regression/StreamingDecay.scala --- @@ -0,0 +1,116 @@ +/* + * Licensed to the Apache So

[GitHub] spark pull request: [SPARK-8952] [SPARKR] - Wrap normalizePath cal...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8343#issuecomment-135595930 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 h

[GitHub] spark pull request: [SPARK-8952] [SPARKR] - Wrap normalizePath cal...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8343#issuecomment-135595941 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 have

[GitHub] spark pull request: [SPARK-8952] [SPARKR] - Wrap normalizePath cal...

2015-08-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8343#issuecomment-135596210 [Test build #41715 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41715/consoleFull) for PR 8343 at commit [`472c767`](https://gith

[GitHub] spark pull request: [SPARK-8952] [SPARKR] - Wrap normalizePath cal...

2015-08-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8343#issuecomment-135598213 [Test build #41715 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41715/console) for PR 8343 at commit [`472c767`](https://github.

[GitHub] spark pull request: [SPARK-8952] [SPARKR] - Wrap normalizePath cal...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8343#issuecomment-135598611 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 project

[GitHub] spark pull request: [SPARK-8952] [SPARKR] - Wrap normalizePath cal...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8343#issuecomment-135598617 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41

[GitHub] spark pull request: [SPARK-8057][Core]Call TaskAttemptContext.getT...

2015-08-27 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/6599#issuecomment-135598700 > I think that we should also backport this to branch-1.4. +1 since we fix it in 1.5.0. Just confirmed this one didn't have conflicts with branch-1.4. --- If y

[GitHub] spark pull request: [SPARK-9986][SPARK-9991][SPARK-9993][SQL]Creat...

2015-08-27 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/8464#discussion_r38164639 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/local/LimitNode.scala --- @@ -0,0 +1,45 @@ +/* +* Licensed to the Apache Software Fou

[GitHub] spark pull request: [SPARK-10049][SPARKR] Support collecting data ...

2015-08-27 Thread sun-rui
Github user sun-rui commented on the pull request: https://github.com/apache/spark/pull/8458#issuecomment-135601195 @davies , @shivaram , Could you help to review 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 you

[GitHub] spark pull request: [SPARK-10326] [yarn] Fix app submission on win...

2015-08-27 Thread vanzin
GitHub user vanzin opened a pull request: https://github.com/apache/spark/pull/8493 [SPARK-10326] [yarn] Fix app submission on windows. You can merge this pull request into a Git repository by running: $ git pull https://github.com/vanzin/spark SPARK-10326 Alternatively you c

[GitHub] spark pull request: [SPARK-10326] [yarn] Fix app submission on win...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8493#issuecomment-135601511 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 have

[GitHub] spark pull request: [SPARK-10326] [yarn] Fix app submission on win...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8493#issuecomment-135601503 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 h

[GitHub] spark pull request: [SPARK-9986][SPARK-9991][SPARK-9993][SQL]Creat...

2015-08-27 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/8464#discussion_r38165165 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/local/LocalNodeTest.scala --- @@ -0,0 +1,189 @@ +/* +* Licensed to the Apache Softwa

[GitHub] spark pull request: [SPARK-10326] [yarn] Fix app submission on win...

2015-08-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8493#issuecomment-135602276 [Test build #41716 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41716/consoleFull) for PR 8493 at commit [`a14dba5`](https://gith

[GitHub] spark pull request: [CORE][MINOR] Whitespace fixes in RangePartiti...

2015-08-27 Thread ihainan
Github user ihainan closed the pull request at: https://github.com/apache/spark/pull/8480 --- 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 ena

[GitHub] spark pull request: [SPARK-10311][Streaming]Reload appId and attem...

2015-08-27 Thread XuTingjun
Github user XuTingjun commented on the pull request: https://github.com/apache/spark/pull/8477#issuecomment-135602834 Sorry that I don't declare the problem clearly. When an app starts with CheckPoint file using [getOrCreate method](https://github.com/apache/spark/blob/master/

[GitHub] spark pull request: [SPARK-10188] [Pyspark] Pyspark CrossValidator...

2015-08-27 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/8399#issuecomment-135603112 Ping @mengxr In case I can't check this soon, it would be great to get this into 1.5 if there is an RC3. --- If your project is set up for it, you can reply to this e

[GitHub] spark pull request: [SPARK-9986][SPARK-9991][SPARK-9993][SQL]Creat...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8464#issuecomment-135603138 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 h

[GitHub] spark pull request: [SPARK-9986][SPARK-9991][SPARK-9993][SQL]Creat...

2015-08-27 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/8464#discussion_r38165742 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/local/LocalNodeTest.scala --- @@ -0,0 +1,189 @@ +/* +* Licensed to the Apache Softwa

[GitHub] spark pull request: [SPARK-10188] [Pyspark] Pyspark CrossValidator...

2015-08-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8399#issuecomment-135603184 [Test build #1700 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1700/consoleFull) for PR 8399 at commit [`bada453`](https://git

[GitHub] spark pull request: [SPARK-9986][SPARK-9991][SPARK-9993][SQL]Creat...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8464#issuecomment-135603152 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 have

[GitHub] spark pull request: [SPARK-9986][SPARK-9991][SPARK-9993][SQL]Creat...

2015-08-27 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/8464#discussion_r38165750 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/local/LimitNode.scala --- @@ -0,0 +1,45 @@ +/* +* Licensed to the Apache Software Fou

[GitHub] spark pull request: [SPARK-9986][SPARK-9991][SPARK-9993][SQL]Creat...

2015-08-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8464#issuecomment-135603859 [Test build #41717 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41717/consoleFull) for PR 8464 at commit [`7dcd502`](https://gith

[GitHub] spark pull request: SPARK-9545, SPARK-9547: Use Maven in PRB if ti...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7878#issuecomment-135604767 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 project

[GitHub] spark pull request: SPARK-9545, SPARK-9547: Use Maven in PRB if ti...

2015-08-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7878#issuecomment-135604744 [Test build #41711 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41711/console) for PR 7878 at commit [`cf58c49`](https://github.

[GitHub] spark pull request: SPARK-9545, SPARK-9547: Use Maven in PRB if ti...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7878#issuecomment-135604771 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41

[GitHub] spark pull request: [SPARK-10188] [Pyspark] Pyspark CrossValidator...

2015-08-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8399#issuecomment-135605403 [Test build #1700 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1700/console) for PR 8399 at commit [`bada453`](https://github

[GitHub] spark pull request: [SPARK-9986][SPARK-9991][SPARK-9993][SQL]Creat...

2015-08-27 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/8464#discussion_r38166499 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/local/LimitNode.scala --- @@ -0,0 +1,45 @@ +/* +* Licensed to the Apache Software Fou

[GitHub] spark pull request: [SPARK-8505][SparkR] Add settings to kick `lin...

2015-08-27 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/7883#issuecomment-135615906 Alright I'm going to merge this as its better to do so before more breaking style changes get in. Will watch Jenkins for the next couple of hours to make sure things ar

[GitHub] spark pull request: [SPARK-8813][SQL] Combine files when there're ...

2015-08-27 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/8125#discussion_r38167285 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/sources/CombineSmallFile.scala --- @@ -0,0 +1,43 @@ +/* + * Licensed to the Apache Softw

[GitHub] spark pull request: [SPARK-8952] [SPARKR] - Wrap normalizePath cal...

2015-08-27 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/8343#issuecomment-135616119 Thanks @lresende -- LGTM --- 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 t

[GitHub] spark pull request: [SPARK-8952] [SPARKR] - Wrap normalizePath cal...

2015-08-27 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/8343#issuecomment-135616345 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 hav

[GitHub] spark pull request: [SPARK-8505][SparkR] Add settings to kick `lin...

2015-08-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/7883 --- 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 enab

[GitHub] spark pull request: [SPARK-8952] [SPARKR] - Wrap normalizePath cal...

2015-08-27 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/8343#issuecomment-135616422 @lresende I'm just retesting this as we merged a R style checker. I'm sure this PR should be fine, but just want to run this through to make sure things are working fin

[GitHub] spark pull request: [SPARK-8952] [SPARKR] - Wrap normalizePath cal...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8343#issuecomment-135616551 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 have

[GitHub] spark pull request: [SPARK-8952] [SPARKR] - Wrap normalizePath cal...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8343#issuecomment-135616536 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 h

[GitHub] spark pull request: [SPARK-8952] [SPARKR] - Wrap normalizePath cal...

2015-08-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8343#issuecomment-135617402 [Test build #41718 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41718/consoleFull) for PR 8343 at commit [`472c767`](https://gith

[GitHub] spark pull request: [SPARK-8505][SparkR] Add settings to kick `lin...

2015-08-27 Thread yu-iskw
Github user yu-iskw commented on the pull request: https://github.com/apache/spark/pull/7883#issuecomment-135617611 @shivaram thank you for merging it. I keep watching the Jenkins in a couple of hours. If it will go well, I will inform the community about this lint script. --- If yo

[GitHub] spark pull request: [SPARK-8952] [SPARKR] - Wrap normalizePath cal...

2015-08-27 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/8343#issuecomment-135617728 @yu-iskw I actually an error in Jenkins which says. I guess this is from the PR that added na.omit to the NAMESPACE yesterday ``` Error in (function (classes, fd

[GitHub] spark pull request: [SPARK-9741][SQL] Approximate Count Distinct u...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8362#issuecomment-135617983 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 h

[GitHub] spark pull request: [SPARK-9741][SQL] Approximate Count Distinct u...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8362#issuecomment-135617993 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 have

[GitHub] spark pull request: [SPARK-8952] [SPARKR] - Wrap normalizePath cal...

2015-08-27 Thread yu-iskw
Github user yu-iskw commented on the pull request: https://github.com/apache/spark/pull/8343#issuecomment-135617969 @shivaram I see. I'll investigate the cause. Thank you for letting me know. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-10065][SQL] Avoid triple copying of var...

2015-08-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8484#issuecomment-135618028 [Test build #41714 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41714/console) for PR 8484 at commit [`35371fb`](https://github.

[GitHub] spark pull request: [SPARK-10327][SQL] Cache Table is not working ...

2015-08-27 Thread chenghao-intel
GitHub user chenghao-intel opened a pull request: https://github.com/apache/spark/pull/8494 [SPARK-10327][SQL] Cache Table is not working while subquery has alias in its project list ```scala import org.apache.spark.sql.hive.execution.HiveTableScan sql("select key, v

[GitHub] spark pull request: [SPARK-10065][SQL] Avoid triple copying of var...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8484#issuecomment-135618198 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41

[GitHub] spark pull request: [SPARK-10065][SQL] Avoid triple copying of var...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8484#issuecomment-135618193 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 project

[GitHub] spark pull request: [SPARK-10327][SQL] Cache Table is not working ...

2015-08-27 Thread chenghao-intel
Github user chenghao-intel commented on the pull request: https://github.com/apache/spark/pull/8494#issuecomment-135618510 cc @marmbrus --- 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 f

[GitHub] spark pull request: [SPARK-9741][SQL] Approximate Count Distinct u...

2015-08-27 Thread hvanhovell
Github user hvanhovell commented on the pull request: https://github.com/apache/spark/pull/8362#issuecomment-135619245 Implemented initial non-sparse HLL++. I am going to take a look at the sparse version next week. The results are still equal to the Clearspring HLL+ implementation in

[GitHub] spark pull request: [SPARK-8952] [SPARKR] - Wrap normalizePath cal...

2015-08-27 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/8343#issuecomment-135619437 I think I found the problem. Our `setGeneric` for `na.omit` is wrong. It is being too restrictive. We need a diff which looks like ``` - function

[GitHub] spark pull request: [SPARK-10327][SQL] Cache Table is not working ...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8494#issuecomment-135619597 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 have

[GitHub] spark pull request: [SPARK-10327][SQL] Cache Table is not working ...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8494#issuecomment-135619546 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 h

[GitHub] spark pull request: [SPARK-9741][SQL] Approximate Count Distinct u...

2015-08-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8362#issuecomment-135619890 [Test build #41719 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41719/consoleFull) for PR 8362 at commit [`1ea722b`](https://gith

[GitHub] spark pull request: [SPARK-10327][SQL] Cache Table is not working ...

2015-08-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8494#issuecomment-135620007 [Test build #41720 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41720/consoleFull) for PR 8494 at commit [`bfd40d9`](https://gith

[GitHub] spark pull request: [SPARK-8952] [SPARKR] - Wrap normalizePath cal...

2015-08-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8343#issuecomment-135620132 [Test build #41718 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41718/console) for PR 8343 at commit [`472c767`](https://github.

[GitHub] spark pull request: [SPARK-8952] [SPARKR] - Wrap normalizePath cal...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8343#issuecomment-135620319 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41

[GitHub] spark pull request: [SPARK-8952] [SPARKR] - Wrap normalizePath cal...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8343#issuecomment-135620315 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 project

[GitHub] spark pull request: [SPARKR] [SPARK-10328] Fix generic for na.omit

2015-08-27 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/8495#issuecomment-135620501 cc @yu-iskw --- 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: [SPARKR] [SPARK-10328] Fix generic for na.omit

2015-08-27 Thread shivaram
GitHub user shivaram opened a pull request: https://github.com/apache/spark/pull/8495 [SPARKR] [SPARK-10328] Fix generic for na.omit S3 function is at https://stat.ethz.ch/R-manual/R-patched/library/stats/html/na.fail.html You can merge this pull request into a Git repository by ru

[GitHub] spark pull request: [SPARK-10323] [SQL] fix nullability of In/InSe...

2015-08-27 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/8492#issuecomment-135620750 Here is the output of some sample tests using hive 1.2.1 ``` hive> select cast(null as string) in ('1', cast(null as string)); OK NULL Time taken: 0.042

[GitHub] spark pull request: [SPARKR] [SPARK-10328] Fix generic for na.omit

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8495#issuecomment-135620819 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 have

[GitHub] spark pull request: [SPARKR] [SPARK-10328] Fix generic for na.omit

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8495#issuecomment-135620803 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 h

[GitHub] spark pull request: [SPARKR] [SPARK-10328] Fix generic for na.omit

2015-08-27 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/8495#issuecomment-135620933 @yu-iskw I also found a minor bug in lint-r that I just fixed. Please let me know if that is good. With this change lint-r passes on my machine --- If your project is

[GitHub] spark pull request: [SPARKR] [SPARK-10328] Fix generic for na.omit

2015-08-27 Thread yu-iskw
Github user yu-iskw commented on the pull request: https://github.com/apache/spark/pull/8495#issuecomment-135621454 @shivaram All right. I am checking this on my local machine. Please give me some minutes. --- If your project is set up for it, you can reply to this email and have you

[GitHub] spark pull request: [SPARKR] [SPARK-10328] Fix generic for na.omit

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8495#issuecomment-135621545 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 h

[GitHub] spark pull request: [SPARK-10323] [SQL] fix nullability of In/InSe...

2015-08-27 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/8492#issuecomment-135621549 postgresql's output regarding `in`... ``` yhuai=# select cast(null as char(10)) in ('1', cast(null as char(10))); ?column? -- (

[GitHub] spark pull request: [SPARKR] [SPARK-10328] Fix generic for na.omit

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8495#issuecomment-135621561 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 have

[GitHub] spark pull request: [SPARKR] [SPARK-10328] Fix generic for na.omit

2015-08-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8495#issuecomment-135621721 [Test build #41722 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41722/consoleFull) for PR 8495 at commit [`4758a87`](https://gith

[GitHub] spark pull request: [SPARKR] [SPARK-10328] Fix generic for na.omit

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8495#issuecomment-135622468 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41

[GitHub] spark pull request: [SPARKR] [SPARK-10328] Fix generic for na.omit

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8495#issuecomment-135622466 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 project

[GitHub] spark pull request: [SPARK-10049][SPARKR] Support collecting data ...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8458#issuecomment-135623380 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 have

[GitHub] spark pull request: [SPARK-10323] [SQL] fix nullability of In/InSe...

2015-08-27 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/8492#issuecomment-135623342 OK. I guess the main question at here is if we want to have a different semantic with hive on `array_contains` regarding `null`. --- If your project is set up for it, you

[GitHub] spark pull request: [SPARK-10049][SPARKR] Support collecting data ...

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8458#issuecomment-135623371 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 h

[GitHub] spark pull request: [SPARKR] [SPARK-10328] Fix generic for na.omit

2015-08-27 Thread yu-iskw
Github user yu-iskw commented on a diff in the pull request: https://github.com/apache/spark/pull/8495#discussion_r38168825 --- Diff: R/pkg/R/generics.R --- @@ -413,7 +413,7 @@ setGeneric("dropna", #' @rdname nafunctions #' @export setGeneric("na.omit", --- End di

[GitHub] spark pull request: [SPARK-5753] [SQL] add JDBCRDD support for pos...

2015-08-27 Thread jakajancar
Github user jakajancar commented on the pull request: https://github.com/apache/spark/pull/4549#issuecomment-135623856 @lepfhty Has any progress been made on this? Can you point me to a PR/branch/JIRA issue/... that I can subscribe to? --- If your project is set up for it, you can re

<    1   2   3   4   5   6   7   >