[GitHub] spark pull request #15864: [SPARK-18420]Fix the compile errors caused by che...

2016-11-12 Thread ConeyLiu
GitHub user ConeyLiu opened a pull request: https://github.com/apache/spark/pull/15864 [SPARK-18420]Fix the compile errors caused by checkstyle ## What changes were proposed in this pull request? Small fix, fix the compile errors caused by checkstyle ## How was

[GitHub] spark pull request #15864: [SPARK-18420]Fix the compile errors caused by che...

2016-11-12 Thread ConeyLiu
Github user ConeyLiu closed the pull request at: https://github.com/apache/spark/pull/15864 --- 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 issue #15863: [SPARK-18419][SQL] Fix `JDBCOptions.asConnectionProperti...

2016-11-12 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15863 Thank you for review, @gatorsmile . In the Spark code, `JDBCOption` is always receiving `CaseInsensitiveMap`. For example, here. -

[GitHub] spark issue #15862: [SPARK-18382][WEBUI] "run at null:-1" in UI when no file...

2016-11-12 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15862 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 feature enabled and

[GitHub] spark issue #15863: [SPARK-18419][SQL] Fix `JDBCOptions.asConnectionProperti...

2016-11-12 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/15863 Not always. Try this code path. ```Scala val df = spark.createDataFrame(sparkContext.parallelize(arr2x2), schema2) df.write.format("jdbc") .option("URL", url1)

[GitHub] spark issue #15659: [SPARK-1267][SPARK-18129] Allow PySpark to be pip instal...

2016-11-12 Thread holdenk
Github user holdenk commented on the issue: https://github.com/apache/spark/pull/15659 So @joshrosen or @davies now that I've updated the shell scripts does this look good to go? I'll make a follow up to maybe add shellcheck to our standard linting if it's something we want

[GitHub] spark pull request #15851: [SPARK-18412][SPARKR][ML] Fix exception for some ...

2016-11-12 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/15851#discussion_r87693230 --- Diff: mllib/src/main/scala/org/apache/spark/ml/r/RWrapperUtils.scala --- @@ -41,5 +41,29 @@ object RWrapperUtils extends Logging {

[GitHub] spark issue #15863: [SPARK-18419][SQL] Fix `JDBCOptions.asConnectionProperti...

2016-11-12 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/15863 Hi @dongjoon-hyun, I just happened to look at this PR and I am just curious. Would this be possible to override `filterKeys` in `CastInsensitiveMap` as something like below? ```scala

[GitHub] spark issue #15861: [SPARK-18294][CORE] Implement commit protocol to support...

2016-11-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15861 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68562/ Test FAILed. ---

[GitHub] spark issue #15861: [SPARK-18294][CORE] Implement commit protocol to support...

2016-11-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15861 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 does not have this feature

[GitHub] spark issue #13837: [SPARK-16126] [SQL] Better Error Message When using Data...

2016-11-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13837 **[Test build #68567 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68567/consoleFull)** for PR 13837 at commit

[GitHub] spark issue #15863: [SPARK-18419][SQL] Fix `JDBCOptions.asConnectionProperti...

2016-11-12 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15863 Oh, sure. I'll investigate 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

[GitHub] spark issue #15866: [SPARK-18422][CORE] Fix wholeTextFiles test to pass on W...

2016-11-12 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/15866 Build started: [CORE] `org.apache.spark.JavaAPISuite`

[GitHub] spark pull request #15863: [SPARK-18419][SQL] Fix `JDBCOptions.asConnectionP...

2016-11-12 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/15863#discussion_r87700361 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala --- @@ -314,8 +311,7 @@ case class DataSource(

[GitHub] spark issue #15860: [SPARK-18418] Fix flags for make_binary_release for hado...

2016-11-12 Thread holdenk
Github user holdenk commented on the issue: https://github.com/apache/spark/pull/15860 re-ping @joshrosen? --- 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,

[GitHub] spark issue #15863: [SPARK-18419][SQL] Fix JDBCOptions and DataSource to be ...

2016-11-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15863 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 does not have this feature

[GitHub] spark issue #15863: [SPARK-18419][SQL] Fix JDBCOptions and DataSource to be ...

2016-11-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15863 **[Test build #68572 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68572/consoleFull)** for PR 15863 at commit

[GitHub] spark issue #15863: [SPARK-18419][SQL] Fix `JDBCOptions.asConnectionProperti...

2016-11-12 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15863 Yes. It seems not to solve this problem. The problem is the following list is case-sensitive. ``` private val jdbcOptionNames = ArrayBuffer.empty[String] ``` --- If your project

[GitHub] spark pull request #15866: [SPARK-18422][CORE] Fix wholeTextFiles test to pa...

2016-11-12 Thread HyukjinKwon
GitHub user HyukjinKwon opened a pull request: https://github.com/apache/spark/pull/15866 [SPARK-18422][CORE] Fix wholeTextFiles test to pass on Windows in JavaAPISuite ## What changes were proposed in this pull request? This PR fixes the test `wholeTextFiles` in

[GitHub] spark issue #15659: [SPARK-1267][SPARK-18129] Allow PySpark to be pip instal...

2016-11-12 Thread holdenk
Github user holdenk commented on the issue: https://github.com/apache/spark/pull/15659 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 have this feature enabled and

[GitHub] spark pull request #15867: [WIP][SPARK-18423][Streaming] ReceiverTracker sho...

2016-11-12 Thread HyukjinKwon
GitHub user HyukjinKwon opened a pull request: https://github.com/apache/spark/pull/15867 [WIP][SPARK-18423][Streaming] ReceiverTracker should close checkpoint dir when stopped even if it was not started ## What changes were proposed in this pull request? Several tests are

[GitHub] spark pull request #14733: [SPARK-17170] [SQL] InMemoryTableScanExec driver-...

2016-11-12 Thread pwoody
Github user pwoody closed the pull request at: https://github.com/apache/spark/pull/14733 --- 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 issue #14733: [SPARK-17170] [SQL] InMemoryTableScanExec driver-side pa...

2016-11-12 Thread pwoody
Github user pwoody commented on the issue: https://github.com/apache/spark/pull/14733 Closing stale PR --- 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

[GitHub] spark issue #15655: [SPARK-18010][CORE] Reduce work performed for building u...

2016-11-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15655 **[Test build #68564 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68564/consoleFull)** for PR 15655 at commit

[GitHub] spark issue #15655: [SPARK-18010][CORE] Reduce work performed for building u...

2016-11-12 Thread vijoshi
Github user vijoshi commented on the issue: https://github.com/apache/spark/pull/15655 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 feature enabled and wishes so,

[GitHub] spark issue #15717: [SPARK-17910][SQL] Allow users to update the comment of ...

2016-11-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15717 **[Test build #68565 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68565/consoleFull)** for PR 15717 at commit

[GitHub] spark issue #15857: [SPARK-18300][SQL] Do not apply foldable propagation wit...

2016-11-12 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/15857 cc @dongjoon-hyun @cloud-fan --- 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 issue #15815: [DOCS][SPARK-18365] Improve Sample Method Documentation

2016-11-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15815 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68566/ Test FAILed. ---

[GitHub] spark issue #15815: [DOCS][SPARK-18365] Improve Sample Method Documentation

2016-11-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15815 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 does not have this feature

[GitHub] spark pull request #15851: [SPARK-18412][SPARKR][ML] Fix exception for some ...

2016-11-12 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/15851#discussion_r87693172 --- Diff: R/pkg/inst/tests/testthat/test_mllib.R --- @@ -971,10 +971,15 @@ test_that("spark.randomForest Classification", { predictions <-

[GitHub] spark issue #15863: [SPARK-18419][SQL] Fix `JDBCOptions.asConnectionProperti...

2016-11-12 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/15863 Oh, it seems not. I just removed my suggestion. I will take a look at this again if it is possible in similar way. Thanks! --- If your project is set up for it, you can reply to this email and

[GitHub] spark issue #15863: [SPARK-18419][SQL] Fix `JDBCOptions.asConnectionProperti...

2016-11-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15863 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 does not have this feature

[GitHub] spark issue #15863: [SPARK-18419][SQL] Fix `JDBCOptions.asConnectionProperti...

2016-11-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15863 **[Test build #68563 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68563/consoleFull)** for PR 15863 at commit

[GitHub] spark issue #15815: [DOCS][SPARK-18365] Documentation is Switched on Sample ...

2016-11-12 Thread anabranch
Github user anabranch commented on the issue: https://github.com/apache/spark/pull/15815 Sounds good to me. I will update it shortly. --- 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 issue #15659: [SPARK-1267][SPARK-18129] Allow PySpark to be pip instal...

2016-11-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15659 **[Test build #68570 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68570/consoleFull)** for PR 15659 at commit

[GitHub] spark issue #15857: [SPARK-18300][SQL] Do not apply foldable propagation wit...

2016-11-12 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15857 Thank you for pining me, @gatorsmile . Thank you for fixing this, @hvanhovell . --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark issue #15862: [SPARK-18382][WEBUI] "run at null:-1" in UI when no file...

2016-11-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15862 **[Test build #68568 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68568/consoleFull)** for PR 15862 at commit

[GitHub] spark issue #15862: [SPARK-18382][WEBUI] "run at null:-1" in UI when no file...

2016-11-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15862 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 does not have this feature

[GitHub] spark issue #15862: [SPARK-18382][WEBUI] "run at null:-1" in UI when no file...

2016-11-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15862 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68568/ Test PASSed. ---

[GitHub] spark pull request #15851: [SPARK-18412][SPARKR][ML] Fix exception for some ...

2016-11-12 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/15851#discussion_r87693194 --- Diff: mllib/src/main/scala/org/apache/spark/ml/r/RWrapperUtils.scala --- @@ -41,5 +41,29 @@ object RWrapperUtils extends Logging {

[GitHub] spark issue #15826: [SPARK-14077][ML][FOLLOW-UP] Minor refactor and cleanup ...

2016-11-12 Thread yanboliang
Github user yanboliang commented on the issue: https://github.com/apache/spark/pull/15826 Merged into master and branch-2.1. Thanks for reviewing. --- 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 #15865: [SPARK-18420]Fix the compile errors caused by che...

2016-11-12 Thread ConeyLiu
GitHub user ConeyLiu opened a pull request: https://github.com/apache/spark/pull/15865 [SPARK-18420]Fix the compile errors caused by checkstyle ## What changes were proposed in this pull request? Small fix, fix the compile errors caused by checkstyle ## How was

[GitHub] spark issue #15694: [SPARK-18179][SQL] Throws analysis exception with a prop...

2016-11-12 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/15694 Would there be other things I should take care of maybe? --- 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 issue #15863: [SPARK-18419][SQL] Fix `JDBCOptions.asConnectionProperti...

2016-11-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15863 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68563/ Test PASSed. ---

[GitHub] spark issue #15815: [DOCS][SPARK-18365] Improve Sample Method Documentation

2016-11-12 Thread anabranch
Github user anabranch commented on the issue: https://github.com/apache/spark/pull/15815 @srowen Think this is probably ready. - [ ] Updated All Languages - [ ] Updated Ticket Description --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark issue #15815: [DOCS][SPARK-18365] Improve Sample Method Documentation

2016-11-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15815 **[Test build #68566 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68566/consoleFull)** for PR 15815 at commit

[GitHub] spark pull request #15717: [SPARK-17910][SQL] Allow users to update the comm...

2016-11-12 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/15717#discussion_r87697174 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala --- @@ -273,6 +273,68 @@ case class AlterTableUnsetPropertiesCommand(

[GitHub] spark pull request #15717: [SPARK-17910][SQL] Allow users to update the comm...

2016-11-12 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/15717#discussion_r87697240 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala --- @@ -273,6 +273,68 @@ case class AlterTableUnsetPropertiesCommand(

[GitHub] spark issue #15717: [SPARK-17910][SQL] Allow users to update the comment of ...

2016-11-12 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/15717 A basic question. Like Hive, we should not support `ALTER VIEW CHANGE COLUMN`, right? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark issue #15815: [DOCS][SPARK-18365] Improve Sample Method Documentation

2016-11-12 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/15815 https://cloud.githubusercontent.com/assets/6477701/20239969/3d5cb0ba-a950-11e6-8a55-e96e8cd02970.png;> For Python documenation, it seems fine. --- If your project is set up for it, you

[GitHub] spark issue #15863: [SPARK-18419][SQL] Fix `JDBCOptions.asConnectionProperti...

2016-11-12 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/15863 Our user-specified JDBC options/parameters are case sensitive, right? --- 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 issue #15659: [SPARK-1267][SPARK-18129] Allow PySpark to be pip instal...

2016-11-12 Thread minrk
Github user minrk commented on the issue: https://github.com/apache/spark/pull/15659 Sounds great! --- 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

[GitHub] spark issue #15815: [DOCS][SPARK-18365] Improve Sample Method Documentation

2016-11-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15815 **[Test build #68566 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68566/consoleFull)** for PR 15815 at commit

[GitHub] spark issue #15866: [SPARK-18422][CORE] Fix wholeTextFiles test to pass on W...

2016-11-12 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/15866 @srowen It seems `org.apache.spark.JavaAPISuite.writeWithNewAPIHadoopFile` is also being failed but the root cause seems different. That is being failed with the exception as below:

[GitHub] spark issue #15815: [DOCS][SPARK-18365] Improve Sample Method Documentation

2016-11-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15815 **[Test build #68573 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68573/consoleFull)** for PR 15815 at commit

[GitHub] spark issue #15863: [SPARK-18419][SQL] Fix `JDBCOptions.asConnectionProperti...

2016-11-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15863 **[Test build #68572 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68572/consoleFull)** for PR 15863 at commit

[GitHub] spark issue #15815: [DOCS][SPARK-18365] Improve Sample Method Documentation

2016-11-12 Thread anabranch
Github user anabranch commented on the issue: https://github.com/apache/spark/pull/15815 The test failure seems quite unrelated but we'll see if it happens again. --- 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 #15815: [DOCS][SPARK-18365] Improve Sample Method Documen...

2016-11-12 Thread anabranch
Github user anabranch commented on a diff in the pull request: https://github.com/apache/spark/pull/15815#discussion_r87699964 --- Diff: R/pkg/R/DataFrame.R --- @@ -936,7 +936,9 @@ setMethod("unique", #' Sample #' -#' Return a sampled subset of this

[GitHub] spark issue #15659: [SPARK-1267][SPARK-18129] Allow PySpark to be pip instal...

2016-11-12 Thread holdenk
Github user holdenk commented on the issue: https://github.com/apache/spark/pull/15659 and or @rxin? --- 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

[GitHub] spark issue #15866: [SPARK-18422][CORE] Fix wholeTextFiles test to pass on W...

2016-11-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15866 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 does not have this feature

[GitHub] spark issue #15863: [SPARK-18419][SQL] Fix JDBCOptions and DataSource to be ...

2016-11-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15863 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68572/ Test FAILed. ---

[GitHub] spark issue #15866: [SPARK-18422][CORE] Fix wholeTextFiles test to pass on W...

2016-11-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15866 **[Test build #68569 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68569/consoleFull)** for PR 15866 at commit

[GitHub] spark issue #15866: [SPARK-18422][CORE] Fix wholeTextFiles test to pass on W...

2016-11-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15866 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68569/ Test PASSed. ---

[GitHub] spark issue #15863: [SPARK-18419][SQL] Fix `JDBCOptions.asConnectionProperti...

2016-11-12 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15863 Thank you for review, @HyukjinKwon . Let me see that. --- 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 issue #15865: [SPARK-18420]Fix the compile errors caused by checkstyle

2016-11-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15865 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 project does not have this

[GitHub] spark issue #14788: [SPARK-17174][SQL] Add the support for TimestampType for...

2016-11-12 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/14788 Hi @rxin, do you mind if I ask what you do think about 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

[GitHub] spark issue #15361: [SPARK-17765][SQL] Support for writing out user-defined ...

2016-11-12 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/15361 ping .. --- 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

[GitHub] spark issue #15655: [SPARK-18010][CORE] Reduce work performed for building u...

2016-11-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15655 **[Test build #68564 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68564/consoleFull)** for PR 15655 at commit

[GitHub] spark issue #15655: [SPARK-18010][CORE] Reduce work performed for building u...

2016-11-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15655 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68564/ Test PASSed. ---

[GitHub] spark issue #15655: [SPARK-18010][CORE] Reduce work performed for building u...

2016-11-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15655 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 does not have this feature

[GitHub] spark issue #15854: [SPARK-18415] [SQL] Weird Plan Output when CTE used in R...

2016-11-12 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/15854 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 feature enabled and wishes

[GitHub] spark issue #15862: [SPARK-18382][WEBUI] "run at null:-1" in UI when no file...

2016-11-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15862 **[Test build #68568 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68568/consoleFull)** for PR 15862 at commit

[GitHub] spark issue #15865: [SPARK-18420]Fix the compile errors caused by checkstyle

2016-11-12 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/15865 Hi @ConeyLiu, It seems ``` [ERROR] src/main/java/org/apache/spark/io/NioBufferedFileInputStream.java:[133] (coding) NoFinalizer: Avoid using finalizer method. ``` is

[GitHub] spark issue #15863: [SPARK-18419][SQL] Fix `JDBCOptions.asConnectionProperti...

2016-11-12 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15863 I updated `DataSource` consistently to use `CaseInsensitiveMap` and am running tests. After testing, this PR will include the followings. - Fix `DataSource` to use

[GitHub] spark issue #15857: [SPARK-18300][SQL] Do not apply foldable propagation wit...

2016-11-12 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15857 It looks good to me. --- 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

[GitHub] spark issue #15858: [SPARK-18417][YARN] Define 'spark.yarn.am.port' in yarn ...

2016-11-12 Thread weiqingy
Github user weiqingy commented on the issue: https://github.com/apache/spark/pull/15858 @srowen Thanks for the review. --- 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 issue #15717: [SPARK-17910][SQL] Allow users to update the comment of ...

2016-11-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15717 **[Test build #68565 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68565/consoleFull)** for PR 15717 at commit

[GitHub] spark issue #15843: [SPARK-18274][ML][PYSPARK] Memory leak in PySpark JavaWr...

2016-11-12 Thread holdenk
Github user holdenk commented on the issue: https://github.com/apache/spark/pull/15843 I'm not so sure about that, we still would want to cleanup the underlying Java reference object on delete if it isn't needed anymore. I think the question is do we want to support shallow copy of

[GitHub] spark issue #15867: [WIP][SPARK-18423][Streaming] ReceiverTracker should clo...

2016-11-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15867 **[Test build #68571 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68571/consoleFull)** for PR 15867 at commit

[GitHub] spark pull request #15826: [SPARK-14077][ML][FOLLOW-UP] Minor refactor and c...

2016-11-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/15826 --- 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 issue #15854: [SPARK-18415] [SQL] Weird Plan Output when CTE used in R...

2016-11-12 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/15854 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 feature enabled and wishes

[GitHub] spark issue #15861: [SPARK-18294][CORE] Implement commit protocol to support...

2016-11-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15861 **[Test build #68562 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68562/consoleFull)** for PR 15861 at commit

[GitHub] spark pull request #15815: [DOCS][SPARK-18365] Improve Sample Documentation

2016-11-12 Thread anabranch
Github user anabranch commented on a diff in the pull request: https://github.com/apache/spark/pull/15815#discussion_r87696862 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala --- @@ -1612,7 +1612,9 @@ class Dataset[T] private[sql]( } /**

[GitHub] spark issue #15717: [SPARK-17910][SQL] Allow users to update the comment of ...

2016-11-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15717 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 does not have this feature

[GitHub] spark issue #15717: [SPARK-17910][SQL] Allow users to update the comment of ...

2016-11-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15717 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68565/ Test FAILed. ---

[GitHub] spark pull request #15815: [DOCS][SPARK-18365] Improve Sample Documentation

2016-11-12 Thread anabranch
Github user anabranch commented on a diff in the pull request: https://github.com/apache/spark/pull/15815#discussion_r87696853 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala --- @@ -1612,7 +1612,9 @@ class Dataset[T] private[sql]( } /**

[GitHub] spark issue #15866: [SPARK-18422][CORE] Fix wholeTextFiles test to pass on W...

2016-11-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15866 **[Test build #68569 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68569/consoleFull)** for PR 15866 at commit

[GitHub] spark issue #13837: [SPARK-16126] [SQL] Better Error Message When using Data...

2016-11-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13837 **[Test build #68567 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68567/consoleFull)** for PR 13837 at commit

[GitHub] spark issue #13837: [SPARK-16126] [SQL] Better Error Message When using Data...

2016-11-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13837 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68567/ Test FAILed. ---

[GitHub] spark issue #15867: [WIP][SPARK-18423][Streaming] ReceiverTracker should clo...

2016-11-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15867 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68571/ Test PASSed. ---

[GitHub] spark issue #15867: [WIP][SPARK-18423][Streaming] ReceiverTracker should clo...

2016-11-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15867 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 does not have this feature

[GitHub] spark issue #15867: [WIP][SPARK-18423][Streaming] ReceiverTracker should clo...

2016-11-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15867 **[Test build #68571 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68571/consoleFull)** for PR 15867 at commit

[GitHub] spark issue #15865: [SPARK-18420]Fix the compile errors caused by checkstyle

2016-11-12 Thread ConeyLiu
Github user ConeyLiu commented on the issue: https://github.com/apache/spark/pull/15865 @HyukjinKwon I'm sorry for that, I've fixed it, please take a look. Thanks a lot. --- 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 issue #15854: [SPARK-18415] [SQL] Weird Plan Output when CTE used in R...

2016-11-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15854 **[Test build #68578 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68578/consoleFull)** for PR 15854 at commit

[GitHub] spark issue #15869: [YARN][DOC] Update Yarn configuration doc

2016-11-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15869 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 project does not have this

[GitHub] spark issue #15854: [SPARK-18415] [SQL] Weird Plan Output when CTE used in R...

2016-11-12 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/15854 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,

[GitHub] spark issue #13837: [SPARK-16126] [SQL] Better Error Message When using Data...

2016-11-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13837 **[Test build #68579 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68579/consoleFull)** for PR 13837 at commit

[GitHub] spark pull request #15869: [YARN][DOC] Update Yarn configuration doc

2016-11-12 Thread weiqingy
GitHub user weiqingy opened a pull request: https://github.com/apache/spark/pull/15869 [YARN][DOC] Update Yarn configuration doc ## What changes were proposed in this pull request? - Add documentation for two yarn configurations: `spark.yarn.report.interval` and

[GitHub] spark issue #15863: [SPARK-18419][SQL] Fix JDBCOptions and DataSource to be ...

2016-11-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15863 **[Test build #68574 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68574/consoleFull)** for PR 15863 at commit

[GitHub] spark issue #15815: [DOCS][SPARK-18365] Improve Sample Method Documentation

2016-11-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15815 **[Test build #68573 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68573/consoleFull)** for PR 15815 at commit

  1   2   3   >