[GitHub] spark issue #13775: [SPARK-16060][SQL] Vectorized Orc reader

2016-11-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13775 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 pull request #16009: [SPARK-18318][ML] ML, Graph 2.1 QA: API: New Scal...

2016-11-24 Thread yanboliang
GitHub user yanboliang opened a pull request: https://github.com/apache/spark/pull/16009 [SPARK-18318][ML] ML, Graph 2.1 QA: API: New Scala APIs, docs ## What changes were proposed in this pull request? API review for 2.1, except ```LSH``` related classes which are still under

[GitHub] spark issue #13775: [SPARK-16060][SQL] Vectorized Orc reader

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

[GitHub] spark issue #13775: [SPARK-16060][SQL] Vectorized Orc reader

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

[GitHub] spark issue #16008: [SPARK-18585][SQL] Use `ev.isNull = "false"` if possible...

2016-11-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16008 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 #16008: [SPARK-18585][SQL] Use `ev.isNull = "false"` if possible...

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

[GitHub] spark issue #16008: [SPARK-18585][SQL] Use `ev.isNull = "false"` if possible...

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

[GitHub] spark issue #15977: [SPARK-18436][SQL] isin causing SQL syntax error with JD...

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

[GitHub] spark issue #15977: [SPARK-18436][SQL] isin causing SQL syntax error with JD...

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

[GitHub] spark issue #15977: [SPARK-18436][SQL] isin causing SQL syntax error with JD...

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

[GitHub] spark issue #15977: [SPARK-18436][SQL] isin causing SQL syntax error with JD...

2016-11-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15977 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 #15977: [SPARK-18436][SQL] isin causing SQL syntax error with JD...

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

[GitHub] spark issue #13775: [SPARK-16060][SQL] Vectorized Orc reader

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

[GitHub] spark issue #13775: [SPARK-16060][SQL] Vectorized Orc reader

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

[GitHub] spark issue #16003: [SPARK-18482][SQL] make sure Spark can access the table ...

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

[GitHub] spark issue #16003: [SPARK-18482][SQL] make sure Spark can access the table ...

2016-11-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16003 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 #16003: [SPARK-18482][SQL] make sure Spark can access the table ...

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

[GitHub] spark issue #15992: [SPARK-18560][CORE][STREAMING] Receiver data can not be ...

2016-11-24 Thread uncleGen
Github user uncleGen commented on the issue: https://github.com/apache/spark/pull/15992 @JoshRosen and @zsxwing waiting for your feedback. --- 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 issue #14136: [SPARK-16282][SQL] Implement percentile SQL function.

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

[GitHub] spark issue #14136: [SPARK-16282][SQL] Implement percentile SQL function.

2016-11-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14136 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 #14136: [SPARK-16282][SQL] Implement percentile SQL function.

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

[GitHub] spark issue #16001: [SPARK-18575][WEB] Keep same style: adjust the position ...

2016-11-24 Thread uncleGen
Github user uncleGen commented on the issue: https://github.com/apache/spark/pull/16001 @srowen waiting for your feedback :D --- 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 #15955: [SPARK-18434][ML][FOLLOWUP] Add checking for setS...

2016-11-24 Thread zhengruifeng
Github user zhengruifeng closed the pull request at: https://github.com/apache/spark/pull/15955 --- 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

[GitHub] spark issue #15955: [SPARK-18434][ML][FOLLOWUP] Add checking for setSolver i...

2016-11-24 Thread yanboliang
Github user yanboliang commented on the issue: https://github.com/apache/spark/pull/15955 After offline discussion with @zhengruifeng , we think the validation in the ```Param``` itself should be better, so other ways (which not calling setter method) of setting params can also

[GitHub] spark issue #15966: [SPARK-18413][SQL][FOLLOW-UP] Use `numPartitions` instea...

2016-11-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15966 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 #15966: [SPARK-18413][SQL][FOLLOW-UP] Use `numPartitions` instea...

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

[GitHub] spark issue #15966: [SPARK-18413][SQL][FOLLOW-UP] Use `numPartitions` instea...

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

[GitHub] spark issue #13775: [SPARK-16060][SQL] Vectorized Orc reader

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

[GitHub] spark issue #16008: [SPARK-18585][SQL] Use `ev.isNull = "false"` if possible...

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

[GitHub] spark pull request #16008: [SPARK-18585][SQL] Use `ev.isNull = "false"` if p...

2016-11-24 Thread ueshin
GitHub user ueshin opened a pull request: https://github.com/apache/spark/pull/16008 [SPARK-18585][SQL] Use `ev.isNull = "false"` if possible for Janino to have a chance to optimize. ## What changes were proposed in this pull request? Janino can optimize `true ? a : b`

[GitHub] spark issue #16007: [SPARK-18583][SQL] Fix nullability of InputFileName.

2016-11-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16007 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 #16003: [SPARK-18482][SQL] make sure Spark can access the table ...

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

[GitHub] spark issue #16007: [SPARK-18583][SQL] Fix nullability of InputFileName.

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

[GitHub] spark issue #16007: [SPARK-18583][SQL] Fix nullability of InputFileName.

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

[GitHub] spark pull request #16000: [SPARK-18537][Web UI]Add a REST api to spark stre...

2016-11-24 Thread ChorPangChan
Github user ChorPangChan commented on a diff in the pull request: https://github.com/apache/spark/pull/16000#discussion_r89563670 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/status/api/v1/JacksonMessageWriter.scala --- @@ -0,0 +1,94 @@ +/* + * Licensed

[GitHub] spark issue #15977: [SPARK-18436][SQL] isin causing SQL syntax error with JD...

2016-11-24 Thread jiangxb1987
Github user jiangxb1987 commented on the issue: https://github.com/apache/spark/pull/15977 Thank you for comment@rxin@srowen@gatorsmile! I'll update that later today! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark issue #14136: [SPARK-16282][SQL] Implement percentile SQL function.

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

[GitHub] spark issue #15966: [SPARK-18413][SQL][FOLLOW-UP] Use `numPartitions` instea...

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

[GitHub] spark pull request #15966: [SPARK-18413][SQL][FOLLOW-UP] Use `numPartitions`...

2016-11-24 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/15966#discussion_r89561319 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcRelationProvider.scala --- @@ -35,11 +35,12 @@ class

[GitHub] spark pull request #15966: [SPARK-18413][SQL][FOLLOW-UP] Use `numPartitions`...

2016-11-24 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/15966#discussion_r89560765 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcRelationProvider.scala --- @@ -35,11 +35,12 @@ class

[GitHub] spark pull request #15966: [SPARK-18413][SQL][FOLLOW-UP] Use `numPartitions`...

2016-11-24 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/15966#discussion_r89560594 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcRelationProvider.scala --- @@ -35,11 +35,12 @@ class

[GitHub] spark pull request #15966: [SPARK-18413][SQL][FOLLOW-UP] Use `numPartitions`...

2016-11-24 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/15966#discussion_r89560507 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcRelationProvider.scala --- @@ -35,11 +35,12 @@ class

[GitHub] spark issue #16007: [SPARK-18583][SQL] Fix nullability of InputFileName.

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

[GitHub] spark pull request #16007: [SPARK-18583][SQL] Fix nullability of InputFileNa...

2016-11-24 Thread ueshin
GitHub user ueshin opened a pull request: https://github.com/apache/spark/pull/16007 [SPARK-18583][SQL] Fix nullability of InputFileName. ## What changes were proposed in this pull request? The nullability of `InputFileName` should be `false`. ## How was this patch

[GitHub] spark issue #15955: [SPARK-18434][ML][FOLLOWUP] Add checking for setSolver i...

2016-11-24 Thread zhengruifeng
Github user zhengruifeng commented on the issue: https://github.com/apache/spark/pull/15955 After offline discussion with @yanboliang , I will update this PR to make solvers more private and include MLPC in this PR. --- If your project is set up for it, you can reply to this email

[GitHub] spark pull request #16004: [STS][SQL]multiple STS running in the same machin...

2016-11-24 Thread tanxinz
Github user tanxinz closed the pull request at: https://github.com/apache/spark/pull/16004 --- 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 #15955: [SPARK-18434][ML][FOLLOWUP] Add checking for setSolver i...

2016-11-24 Thread zhengruifeng
Github user zhengruifeng commented on the issue: https://github.com/apache/spark/pull/15955 @yanboliang I add the `private[regression]` limitation refering to `MultilayerPerceptronClassifier` implementation:

[GitHub] spark pull request #16000: [SPARK-18537][Web UI]Add a REST api to spark stre...

2016-11-24 Thread ChorPangChan
Github user ChorPangChan commented on a diff in the pull request: https://github.com/apache/spark/pull/16000#discussion_r89558292 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/status/api/v1/JacksonMessageWriter.scala --- @@ -0,0 +1,94 @@ +/* + * Licensed

[GitHub] spark issue #14995: [Test Only][SPARK-6235][CORE]Address various 2G limits

2016-11-24 Thread witgo
Github user witgo commented on the issue: https://github.com/apache/spark/pull/14995 Thanks, I am glad to hear this, and I want to solve the issues of reading, storing and transmitting data as much as possible. --- If your project is set up for it, you can reply to this email and

[GitHub] spark issue #15916: [SPARK-18487][SQL] Add completion listener to HashAggreg...

2016-11-24 Thread viirya
Github user viirya commented on the issue: https://github.com/apache/spark/pull/15916 Hello @rxin @hvanhovell Can you let me know if this is needed or not? So I can close it. Thank you. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark issue #15970: [SPARK-18134][SQL] Comparable MapTypes [POC]

2016-11-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15970 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 #15970: [SPARK-18134][SQL] Comparable MapTypes [POC]

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

[GitHub] spark issue #15970: [SPARK-18134][SQL] Comparable MapTypes [POC]

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

[GitHub] spark issue #15961: [SPARK-18523][PySpark]Make SparkContext.stop more reliab...

2016-11-24 Thread kxepal
Github user kxepal commented on the issue: https://github.com/apache/spark/pull/15961 @holdenk @srowen Added a warning message, please take a second look. --- 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 #15966: [SPARK-18413][SQL][FOLLOW-UP] Use `numPartitions`...

2016-11-24 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/15966#discussion_r89554371 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcRelationProvider.scala --- @@ -35,11 +35,12 @@ class

[GitHub] spark issue #15966: [SPARK-18413][SQL][FOLLOW-UP] Use `numPartitions` instea...

2016-11-24 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/15966 LGTM other than that one change. --- 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 #15970: [SPARK-18134][SQL] Comparable MapTypes [POC]

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

[GitHub] spark issue #15462: [SPARK-17680] [SQL] [TEST] Added test cases for InMemory...

2016-11-24 Thread kiszk
Github user kiszk commented on the issue: https://github.com/apache/spark/pull/15462 @andrewor14 thank you for your review. Could you please see it 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 project

[GitHub] spark pull request #15358: [SPARK-17783] [SQL] Hide Credentials in CREATE an...

2016-11-24 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/15358#discussion_r89547862 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/ddl.scala --- @@ -56,6 +56,14 @@ case class CreateTempViewUsing(

[GitHub] spark pull request #15358: [SPARK-17783] [SQL] Hide Credentials in CREATE an...

2016-11-24 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/15358#discussion_r89547695 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala --- @@ -766,4 +766,5 @@ object DDLUtils { } }

[GitHub] spark pull request #15358: [SPARK-17783] [SQL] Hide Credentials in CREATE an...

2016-11-24 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/15358#discussion_r89547692 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala --- @@ -52,12 +52,10 @@ case class CatalogStorageFormat(

[GitHub] spark issue #15358: [SPARK-17783] [SQL] Hide Credentials in CREATE and DESC ...

2016-11-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15358 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 #15358: [SPARK-17783] [SQL] Hide Credentials in CREATE and DESC ...

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

[GitHub] spark issue #15358: [SPARK-17783] [SQL] Hide Credentials in CREATE and DESC ...

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

[GitHub] spark issue #16004: [STS][SQL]multiple STS running in the same machine throw...

2016-11-24 Thread hvanhovell
Github user hvanhovell commented on the issue: https://github.com/apache/spark/pull/16004 Hmmm. Seems that you have opened against the wrong branch. Could you close, open against the correct branch, and add the JIRA ticket and the Spark component to the title (look at every other

[GitHub] spark issue #16001: [SPARK-18575][WEB] Keep same style: adjust the position ...

2016-11-24 Thread ajbozarth
Github user ajbozarth commented on the issue: https://github.com/apache/spark/pull/16001 Not sure this is worth a whole commit, but it is probably good to fix. 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

[GitHub] spark issue #15975: [SPARK-18538] [SQL] Fix Concurrent Table Fetching Using ...

2016-11-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15975 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 #15975: [SPARK-18538] [SQL] Fix Concurrent Table Fetching Using ...

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

[GitHub] spark issue #15975: [SPARK-18538] [SQL] Fix Concurrent Table Fetching Using ...

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

[GitHub] spark issue #15462: [SPARK-17680] [SQL] [TEST] Added test cases for InMemory...

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

[GitHub] spark issue #15462: [SPARK-17680] [SQL] [TEST] Added test cases for InMemory...

2016-11-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15462 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 #15462: [SPARK-17680] [SQL] [TEST] Added test cases for InMemory...

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

[GitHub] spark pull request #16005: [SPARK-18578][SQL] Full outer join in correlated ...

2016-11-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/16005 --- 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 #16005: [SPARK-18578][SQL] Full outer join in correlated subquer...

2016-11-24 Thread hvanhovell
Github user hvanhovell commented on the issue: https://github.com/apache/spark/pull/16005 Hmmm. I cannot merge to 2.0 :(... Can you open a backport against 2.0? --- 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 #16005: [SPARK-18578][SQL] Full outer join in correlated subquer...

2016-11-24 Thread hvanhovell
Github user hvanhovell commented on the issue: https://github.com/apache/spark/pull/16005 LGTM. Merging to master/2.1/2.0. Thanks! --- 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 #15977: [SPARK-18436][SQL] isin causing SQL syntax error with JD...

2016-11-24 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/15977 Also use NonFoldableLiteral to do the test. Make sure you create a null nonfoldable literal to verify the result is null. --- If your project is set up for it, you can reply to this email and have

[GitHub] spark issue #15977: [SPARK-18436][SQL] isin causing SQL syntax error with JD...

2016-11-24 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/15977 How about adding the following two lines into the test case in

[GitHub] spark pull request #15977: [SPARK-18436][SQL] isin causing SQL syntax error ...

2016-11-24 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/15977#discussion_r89540400 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JDBCRDD.scala --- @@ -120,7 +120,9 @@ object JDBCRDD extends Logging {

[GitHub] spark issue #15913: [SPARK-18481][ML] ML 2.1 QA: Remove deprecated methods f...

2016-11-24 Thread jkbradley
Github user jkbradley commented on the issue: https://github.com/apache/spark/pull/15913 One more thing: This breaks several public APIs, although in mostly minor ways and with good reasons. Could you please update the JIRA to list the public API changes? --- If your project is

[GitHub] spark pull request #15913: [SPARK-18481][ML] ML 2.1 QA: Remove deprecated me...

2016-11-24 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/15913#discussion_r89537340 --- Diff: mllib/src/main/scala/org/apache/spark/ml/param/params.scala --- @@ -547,21 +547,6 @@ trait Params extends Identifiable with Serializable {

[GitHub] spark pull request #15913: [SPARK-18481][ML] ML 2.1 QA: Remove deprecated me...

2016-11-24 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/15913#discussion_r89537785 --- Diff: mllib/src/main/scala/org/apache/spark/ml/tree/treeParams.scala --- @@ -443,12 +429,11 @@ private[ml] trait GBTParams extends TreeEnsembleParams

[GitHub] spark pull request #15913: [SPARK-18481][ML] ML 2.1 QA: Remove deprecated me...

2016-11-24 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/15913#discussion_r89537864 --- Diff: mllib/src/main/scala/org/apache/spark/ml/tree/treeParams.scala --- @@ -317,8 +317,32 @@ private[ml] trait TreeEnsembleParams extends

[GitHub] spark pull request #15913: [SPARK-18481][ML] ML 2.1 QA: Remove deprecated me...

2016-11-24 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/15913#discussion_r89537771 --- Diff: mllib/src/main/scala/org/apache/spark/ml/tree/treeParams.scala --- @@ -443,12 +425,11 @@ private[ml] trait GBTParams extends TreeEnsembleParams

[GitHub] spark issue #16005: [SPARK-18578][SQL] Full outer join in correlated subquer...

2016-11-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16005 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 #16005: [SPARK-18578][SQL] Full outer join in correlated subquer...

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

[GitHub] spark issue #16005: [SPARK-18578][SQL] Full outer join in correlated subquer...

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

[GitHub] spark issue #15977: [SPARK-18436][SQL] isin causing SQL syntax error with JD...

2016-11-24 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/15977 I believe the correct replacement for "x in ()" is "if (isnull(x)) null else false" --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark issue #15358: [SPARK-17783] [SQL] Hide Credentials in CREATE and DESC ...

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

[GitHub] spark issue #15975: [SPARK-18538] [SQL] Fix Concurrent Table Fetching Using ...

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

[GitHub] spark issue #15975: [SPARK-18538] [SQL] Fix Concurrent Table Fetching Using ...

2016-11-24 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/15975 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 #15717: [SPARK-17910][SQL] Allow users to update the comment of ...

2016-11-24 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/15717 Thanks! --- 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 #15358: [SPARK-17783] [SQL] Hide Credentials in CREATE and DESC ...

2016-11-24 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/15358 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 #16005: [SPARK-18578][SQL] Full outer join in correlated subquer...

2016-11-24 Thread nsyca
Github user nsyca commented on the issue: https://github.com/apache/spark/pull/16005 @hvanhovell I will work on the whitelist in a new JIRA under SPARK-18455. It will be my top priority task and I hope we can merge it in the next minor release of 2.0.x. Let's have this PR scoped for

[GitHub] spark issue #14995: [Test Only][SPARK-6235][CORE]Address various 2G limits

2016-11-24 Thread opme
Github user opme commented on the issue: https://github.com/apache/spark/pull/14995 @witgo I have a Pyspark application that was failing in 3 different places but is able to run without errors now. I'm glad for this patch as I am not sure how I would have explained to my professors

[GitHub] spark issue #16005: [SPARK-18578][SQL] Full outer join in correlated subquer...

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

[GitHub] spark issue #16005: [SPARK-18578][SQL] Full outer join in correlated subquer...

2016-11-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16005 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 #16005: [SPARK-18578][SQL] Full outer join in correlated subquer...

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

[GitHub] spark issue #15462: [SPARK-17680] [SQL] [TEST] Added test cases for InMemory...

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

[GitHub] spark pull request #15462: [SPARK-17680] [SQL] [TEST] Added test cases for I...

2016-11-24 Thread kiszk
Github user kiszk commented on a diff in the pull request: https://github.com/apache/spark/pull/15462#discussion_r89535856 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/columnar/InMemoryColumnarQuerySuite.scala --- @@ -20,18 +20,83 @@ package

[GitHub] spark pull request #15462: [SPARK-17680] [SQL] [TEST] Added test cases for I...

2016-11-24 Thread kiszk
Github user kiszk commented on a diff in the pull request: https://github.com/apache/spark/pull/15462#discussion_r89535786 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/columnar/InMemoryColumnarQuerySuite.scala --- @@ -20,18 +20,83 @@ package

  1   2   3   >