[GitHub] spark pull request: [SPARK-12480][SQL] add Hash expression that ca...

2015-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10435#issuecomment-167338506 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-12480][SQL] add Hash expression that ca...

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

[GitHub] spark pull request: [SPARK-12520] [PySpark] [1.5] Ensure the join ...

2015-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10484#issuecomment-167354765 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-12520] [PySpark] [1.5] Ensure the join ...

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

[GitHub] spark pull request: [SPARK-12218] Fixes ORC conjunction predicate ...

2015-12-26 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/10377#discussion_r48446551 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/orc/OrcSourceSuite.scala --- @@ -174,4 +175,33 @@ class OrcSourceSuite extends OrcSuite {

[GitHub] spark pull request: [SPARK-12526][SPARKR]`ifelse`, `when`, `otherw...

2015-12-26 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/10481#issuecomment-167330045 Thanks @saurfang for the PR. I think this is a good enough fix as this is in an internal implementation detail and not visible to SparkR users ? cc @sun-rui

[GitHub] spark pull request: [SPARK-12480][SQL] add Hash expression that ca...

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

[GitHub] spark pull request: [SPARK-12520] [PySpark] [1.5] Ensure the join ...

2015-12-26 Thread gatorsmile
GitHub user gatorsmile opened a pull request: https://github.com/apache/spark/pull/10484 [SPARK-12520] [PySpark] [1.5] Ensure the join type is `inner` for equi-Join. This PR is to add `assert` to ensure the join type is `inner` for equi-Join. JIRA:

[GitHub] spark pull request: [SPARK-12520] [PySpark] Correct Descriptions a...

2015-12-26 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/10477#issuecomment-167352929 @davies @JoshRosen Could you check if this PR is appropriate? Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-12520] [PySpark] [1.5] Ensure the join ...

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

[GitHub] spark pull request: [SPARK-12520] [PySpark] [1.5] Ensure the join ...

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

[GitHub] spark pull request: [SPARK-12527] Add private val after @transient...

2015-12-26 Thread ted-yu
Github user ted-yu closed the pull request at: https://github.com/apache/spark/pull/10482 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] spark pull request: [SPARK-12218] Fixes ORC conjunction predicate ...

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

[GitHub] spark pull request: [SPARK-12480][SQL] add Hash expression that ca...

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

[GitHub] spark pull request: [WIP][SPARK-9686][SQL] Redirect JDBC metadata ...

2015-12-26 Thread KaiXinXiaoLei
Github user KaiXinXiaoLei commented on a diff in the pull request: https://github.com/apache/spark/pull/9911#discussion_r48445836 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/client/ClientInterface.scala --- @@ -189,4 +191,6 @@ private[hive] trait ClientInterface {

[GitHub] spark pull request: [SPARK-12480][SQL] add Hash expression that ca...

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

[GitHub] spark pull request: [SPARK-12480][SQL] add Hash expression that ca...

2015-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10435#issuecomment-167329445 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [WIP][SPARK-9686][SQL] Redirect JDBC metadata ...

2015-12-26 Thread KaiXinXiaoLei
Github user KaiXinXiaoLei commented on the pull request: https://github.com/apache/spark/pull/9911#issuecomment-167294826 @liancheng I use your code and build. Then run "sbin/start-thriftserver.sh" failed. the error info is as fellow: 15/12/26 16:28:27 INFO ClientWrapper: Loaded

[GitHub] spark pull request: [SPARK-12527] Add private val after @transient...

2015-12-26 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/10482#issuecomment-167294802 Hey @ted-yu, I think this duplicates #10479, my own PR for this issue. Also note that this is not a release-blocker since it's not a regression. --- If your project

[GitHub] spark pull request: [SPARK-12263][Docs]: IllegalStateException: Me...

2015-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10483#issuecomment-167297486 Can one of the admins verify this patch? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-12527] Add private val after @transient...

2015-12-26 Thread ted-yu
Github user ted-yu commented on the pull request: https://github.com/apache/spark/pull/10482#issuecomment-167309902 I noticed the Jenkins sbt build jobs against Scala 2.11 were recreated yesterday. Do you know when the previous 'SPARK-branch-1.5-COMPILE-SBT-SCALA-2.11'

[GitHub] spark pull request: [SPARK-12218] Fixes ORC conjunction predicate ...

2015-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10377#issuecomment-167321193 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-12218] Fixes ORC conjunction predicate ...

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

[GitHub] spark pull request: [SPARK-12526][SPARKR]`ifelse`, `when`, `otherw...

2015-12-26 Thread saurfang
Github user saurfang commented on the pull request: https://github.com/apache/spark/pull/10481#issuecomment-167376718 Indeed. Like I said I don't see compelling reason to use these three functions in vectorized way. Let me know if you have any other comments on the fix. --- If

[GitHub] spark pull request: Branch 1.2

2015-12-26 Thread ErlangZ
GitHub user ErlangZ opened a pull request: https://github.com/apache/spark/pull/10485 Branch 1.2 You can merge this pull request into a Git repository by running: $ git pull https://github.com/apache/spark branch-1.2 Alternatively you can review and apply these changes as

[GitHub] spark pull request: [SPARK-12317][SQL]Support configurable value i...

2015-12-26 Thread kevinyu98
Github user kevinyu98 commented on a diff in the pull request: https://github.com/apache/spark/pull/10314#discussion_r48451909 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLConf.scala --- @@ -100,6 +101,33 @@ private[spark] object SQLConf { } },

[GitHub] spark pull request: [SPARK-12317][SQL]Support configurable value i...

2015-12-26 Thread kevinyu98
Github user kevinyu98 commented on a diff in the pull request: https://github.com/apache/spark/pull/10314#discussion_r48451916 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLConf.scala --- @@ -100,6 +101,33 @@ private[spark] object SQLConf { } },

[GitHub] spark pull request: [SPARK-12317][SQL]Support configurable value i...

2015-12-26 Thread kevinyu98
Github user kevinyu98 commented on a diff in the pull request: https://github.com/apache/spark/pull/10314#discussion_r48451915 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLConf.scala --- @@ -100,6 +101,33 @@ private[spark] object SQLConf { } },

[GitHub] spark pull request: [SPARK-12317][SQL]Support configurable value i...

2015-12-26 Thread kevinyu98
Github user kevinyu98 commented on a diff in the pull request: https://github.com/apache/spark/pull/10314#discussion_r48451911 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLConf.scala --- @@ -100,6 +101,33 @@ private[spark] object SQLConf { } },

[GitHub] spark pull request: [SPARK-12317][SQL]Support configurable value i...

2015-12-26 Thread kevinyu98
Github user kevinyu98 commented on a diff in the pull request: https://github.com/apache/spark/pull/10314#discussion_r48451918 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLConf.scala --- @@ -100,6 +101,33 @@ private[spark] object SQLConf { } },

[GitHub] spark pull request: [SPARK-11460][SQL]Add ExpressionDescription to...

2015-12-26 Thread wilson888888888
GitHub user wilson8 opened a pull request: https://github.com/apache/spark/pull/10486 [SPARK-11460][SQL]Add ExpressionDescription to aggregate functions You can merge this pull request into a Git repository by running: $ git pull

[GitHub] spark pull request: [SPARK-11460][SQL]Add ExpressionDescription to...

2015-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10486#issuecomment-167386674 Can one of the admins verify this patch? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-11460][SQL]Add ExpressionDescription to...

2015-12-26 Thread vectorijk
Github user vectorijk commented on the pull request: https://github.com/apache/spark/pull/10486#issuecomment-167391678 In order to linking this PR to JIRA correctly, you should change title to `[SPARK-12460] ...` rather than `11460`. It is also better to change commit title

[GitHub] spark pull request: Branch 1.2

2015-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10485#issuecomment-167378865 Can one of the admins verify this patch? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-12317][SQL]Support configurable value i...

2015-12-26 Thread kevinyu98
Github user kevinyu98 commented on a diff in the pull request: https://github.com/apache/spark/pull/10314#discussion_r48451947 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLConf.scala --- @@ -107,7 +135,7 @@ private[spark] object SQLConf { isPublic: Boolean