[GitHub] spark pull request: [SPARK-6379][SQL] Support a functon to call us...

2015-03-25 Thread maropu
Github user maropu commented on a diff in the pull request: https://github.com/apache/spark/pull/5061#discussion_r27191782 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala --- @@ -212,6 +212,22 @@ class SQLContext(@transient val sparkContext: SparkContext)

[GitHub] spark pull request: [SPARK-6396][Core] Add broadcast timeout

2015-03-25 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5197#discussion_r27191779 --- Diff: core/src/main/scala/org/apache/spark/network/BlockTransferService.scala --- @@ -99,7 +104,7 @@ abstract class BlockTransferService extends ShuffleClie

[GitHub] spark pull request: [SPARK-6547][Tests]Missing import Files in Ins...

2015-03-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5200#issuecomment-86363846 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 pro

[GitHub] spark pull request: [SPARK-6547][Tests]Missing import Files in Ins...

2015-03-25 Thread zzcclp
GitHub user zzcclp opened a pull request: https://github.com/apache/spark/pull/5200 [SPARK-6547][Tests]Missing import Files in InsertIntoHiveTableSuite.scala Use org.apache.spark.util.Utils instead of Files You can merge this pull request into a Git repository by running: $ git

[GitHub] spark pull request: [SPARK-6546][Build] Using the wrong code that ...

2015-03-25 Thread DoingDone9
Github user DoingDone9 commented on the pull request: https://github.com/apache/spark/pull/5198#issuecomment-86363210 you are right @liancheng --- 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-5885][MLLIB] Add VectorAssembler as a f...

2015-03-25 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/5196#issuecomment-86361537 The build failure was caused by codegen. When the `ExpressionEvaluationSuite.CreateStruct` test case is executed separately, codegen is not enabled by default, and `Cre

[GitHub] spark pull request: [SPARK-6379][SQL] Support a functon to call us...

2015-03-25 Thread maropu
Github user maropu commented on the pull request: https://github.com/apache/spark/pull/5061#issuecomment-86361279 I'll fix soon. --- 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 e

[GitHub] spark pull request: [SPARK-5885][MLLIB] Add VectorAssembler as a f...

2015-03-25 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/5196#discussion_r27191305 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypes.scala --- @@ -142,3 +142,29 @@ case class CreateArray(children

[GitHub] spark pull request: [SPARK-5885][MLLIB] Add VectorAssembler as a f...

2015-03-25 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/5196#discussion_r27191304 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ExpressionEvaluationSuite.scala --- @@ -1080,4 +1080,11 @@ class Expression

[GitHub] spark pull request: [SPARK-5885][MLLIB] Add VectorAssembler as a f...

2015-03-25 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/5196#discussion_r27191279 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypes.scala --- @@ -142,3 +142,29 @@ case class CreateArray(children

[GitHub] spark pull request: [SPARK-6350][Mesos] Make mesosExecutorCores co...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5063#issuecomment-86357920 [Test build #29215 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29215/consoleFull) for PR 5063 at commit [`223aed9`](https://githu

[GitHub] spark pull request: [SPARK-6350][Mesos] Make mesosExecutorCores co...

2015-03-25 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/spark/pull/5063#issuecomment-86357586 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

[GitHub] spark pull request: [SPARK-6542][SQL] add CreateStruct

2015-03-25 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/5195#issuecomment-86357445 The build failure was caused by codegen. Some test suite executed earlier enabled codegen, thus `CreateStruct.dataType` got called somewhere and caused the build failur

[GitHub] spark pull request: [SPARK-6350][Mesos] Make mesosExecutorCores co...

2015-03-25 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/spark/pull/5063#issuecomment-86357482 @sryza Who Do you know help this build issue? I just edited docs only. --- If your project is set up for it, you can reply to this email and have your reply appear on Gi

[GitHub] spark pull request: [SPARK-4897] [PySpark] Python 3 support

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

[GitHub] spark pull request: [SPARK-4897] [PySpark] Python 3 support

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5173#issuecomment-86356829 [Test build #29213 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29213/consoleFull) for PR 5173 at commit [`a716d34`](https://gith

[GitHub] spark pull request: [SPARK-6350][Mesos] Make mesosExecutorCores co...

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

[GitHub] spark pull request: [SPARK-6350][Mesos] Make mesosExecutorCores co...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5063#issuecomment-86356693 [Test build #29214 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29214/consoleFull) for PR 5063 at commit [`223aed9`](https://gith

[GitHub] spark pull request: [SPARK-6545][SQL][WIP] Minor changes

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

[GitHub] spark pull request: [SPARK-6545][SQL][WIP] Minor changes

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5199#issuecomment-86355111 [Test build #29210 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29210/consoleFull) for PR 5199 at commit [`645b9be`](https://gith

[GitHub] spark pull request: [SPARK-6350][Mesos] Make mesosExecutorCores co...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5063#issuecomment-86354447 [Test build #29212 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29212/consoleFull) for PR 5063 at commit [`5c82f96`](https://gith

[GitHub] spark pull request: [SPARK-6350][Mesos] Make mesosExecutorCores co...

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

[GitHub] spark pull request: [SPARK-6451][SQL] supported code generation fo...

2015-03-25 Thread gvramana
Github user gvramana commented on the pull request: https://github.com/apache/spark/pull/5138#issuecomment-86350832 Test Failure does not look related to fix. Please check. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] spark pull request: [SPARK-6546][Build] Using the wrong code that ...

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

[GitHub] spark pull request: [SPARK-6546][Build] Using the wrong code that ...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5198#issuecomment-86348547 [Test build #29206 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29206/consoleFull) for PR 5198 at commit [`6e0140d`](https://gith

[GitHub] spark pull request: [SPARK-6145][SQL] fix ORDER BY on nested field...

2015-03-25 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/5189#issuecomment-86346279 Another problem, for something like `SELECT DISTINCT a FROM t ORDER BY a`, we can't resolve this `Sort` because its child is not `Project` or `Aggregate`, thus can't ha

[GitHub] spark pull request: [SPARK-4897] [PySpark] Python 3 support

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5173#issuecomment-86345793 [Test build #29213 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29213/consoleFull) for PR 5173 at commit [`a716d34`](https://githu

[GitHub] spark pull request: [SPARK-6350][Mesos] Make mesosExecutorCores co...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5063#issuecomment-86345756 [Test build #29214 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29214/consoleFull) for PR 5063 at commit [`223aed9`](https://githu

[GitHub] spark pull request: [SPARK-6350][Mesos] Make mesosExecutorCores co...

2015-03-25 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/spark/pull/5063#issuecomment-86345441 I've rebased from master. Don't mind that I have a mistake to write a commit log which I've worked another issue. --- If your project is set up for it, you can reply to

[GitHub] spark pull request: [SPARK-4897] [PySpark] Python 3 support

2015-03-25 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/5173#issuecomment-86345124 @JoshRosen I had changed to use python3.4, hope that it will work. The PyPy is not the expected version, it's PyPy3 2.4, it should be PyPy 2.4 or 2.5. --- If you

[GitHub] spark pull request: [SPARK-6350][Mesos] Make mesosExecutorCores co...

2015-03-25 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/spark/pull/5063#issuecomment-86344972 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

[GitHub] spark pull request: [SPARK-6350][Mesos] Make mesosExecutorCores co...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5063#issuecomment-86343836 [Test build #29212 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29212/consoleFull) for PR 5063 at commit [`5c82f96`](https://githu

[GitHub] spark pull request: [SPARK-6145][SQL] fix ORDER BY on nested field...

2015-03-25 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/5189#discussion_r27188497 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -340,19 +340,16 @@ class Analyzer(catalog: Catalog,

[GitHub] spark pull request: [SPARK-6443][WIP]Could not submit app in stand...

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

[GitHub] spark pull request: [SPARK-6443][WIP]Could not submit app in stand...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5116#issuecomment-86342173 [Test build #29205 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29205/consoleFull) for PR 5116 at commit [`7a881b3`](https://gith

[GitHub] spark pull request: [SPARK-6350][Mesos] Make mesosExecutorCores co...

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

[GitHub] spark pull request: [SPARK-6350][Mesos] Make mesosExecutorCores co...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5063#issuecomment-86341311 [Test build #29204 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29204/consoleFull) for PR 5063 at commit [`49811dd`](https://gith

[GitHub] spark pull request: [SPARK-6545][SQL][WIP] Minor changes

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

[GitHub] spark pull request: [SPARK-6545][SQL][WIP] Minor changes

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5199#issuecomment-86340564 [Test build #29203 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29203/consoleFull) for PR 5199 at commit [`84d15d5`](https://gith

[GitHub] spark pull request: [SPARK-6546][Build] Using the wrong code that ...

2015-03-25 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/5198#issuecomment-86339048 @JoshRosen @marmbrus #4289 was using Guava's `com.google.common.io.Files` according to the first commit of that PR, see [here] [1]. However, #5029 was merged earlier, a

[GitHub] spark pull request: [SPARK-4897] [PySpark] Python 3 support

2015-03-25 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/5173#issuecomment-86338604 @davies It looks like the Python 3.4 executable on the staging Jenkins is `/usr/local/bin/python3.4`, but our tests expect `python3` in the path. I tried creating a sy

[GitHub] spark pull request: [WIP][SPARK-3468] WebUI Timeline-View feature

2015-03-25 Thread sarutak
Github user sarutak commented on the pull request: https://github.com/apache/spark/pull/2342#issuecomment-86337370 This is an image of "Task Assignment Timeline" view. It's almost done. This view is integrated into existing WebUI, zoomable and scrollable. ![2015-03-25

[GitHub] spark pull request: [SPARK-4123][Project Infra][WIP]: Show new dep...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5093#issuecomment-86337108 [Test build #29211 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29211/consoleFull) for PR 5093 at commit [`5375ad8`](https://githu

[GitHub] spark pull request: [SPARK-4123][Project Infra][WIP]: Show new dep...

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

[GitHub] spark pull request: [SPARK-4123][Project Infra][WIP]: Show new dep...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5093#issuecomment-86337109 [Test build #29211 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29211/consoleFull) for PR 5093 at commit [`5375ad8`](https://gith

[GitHub] spark pull request: [SPARK-6545][SQL][WIP] Minor changes

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5199#issuecomment-86337114 [Test build #29210 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29210/consoleFull) for PR 5199 at commit [`645b9be`](https://githu

[GitHub] spark pull request: [SPARK-4123][Project Infra][WIP]: Show new dep...

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

[GitHub] spark pull request: [SPARK-4123][Project Infra][WIP]: Show new dep...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5093#issuecomment-86336533 [Test build #29209 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29209/consoleFull) for PR 5093 at commit [`9bce980`](https://githu

[GitHub] spark pull request: [SPARK-4123][Project Infra][WIP]: Show new dep...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5093#issuecomment-86336534 [Test build #29209 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29209/consoleFull) for PR 5093 at commit [`9bce980`](https://gith

[GitHub] spark pull request: [SPARK-4123][Project Infra][WIP]: Show new dep...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5093#issuecomment-86334455 [Test build #29208 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29208/consoleFull) for PR 5093 at commit [`8f3c4b4`](https://githu

[GitHub] spark pull request: [SPARK-4123][Project Infra][WIP]: Show new dep...

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

[GitHub] spark pull request: [SPARK-4123][Project Infra][WIP]: Show new dep...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5093#issuecomment-86334459 [Test build #29208 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29208/consoleFull) for PR 5093 at commit [`8f3c4b4`](https://gith

[GitHub] spark pull request: [SPARK-6396][Core] Add broadcast timeout

2015-03-25 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/5197#issuecomment-86333587 Maybe we should document the new configuration in `configuration.md`. --- If your project is set up for it, you can reply to this email and have your reply appear on Gi

[GitHub] spark pull request: [SPARK-6546][Build] Using the wrong code that ...

2015-03-25 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/5198#issuecomment-86331305 This LGTM pending Jenkins. #4289 somehow passed Jenkins, however the build history has been cleaned now. --- If your project is set up for it, you can reply to this e

[GitHub] spark pull request: [SPARK-6396][Core] Add broadcast timeout

2015-03-25 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/5197#discussion_r27187370 --- Diff: core/src/main/scala/org/apache/spark/network/BlockTransferService.scala --- @@ -83,7 +83,12 @@ abstract class BlockTransferService extends Shuffl

[GitHub] spark pull request: [SPARK-4123][Project Infra][WIP]: Show new dep...

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

[GitHub] spark pull request: [SPARK-6396][Core] Add broadcast timeout

2015-03-25 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/5197#discussion_r27187347 --- Diff: core/src/main/scala/org/apache/spark/network/BlockTransferService.scala --- @@ -99,7 +104,7 @@ abstract class BlockTransferService extends Shuffl

[GitHub] spark pull request: [SPARK-4123][Project Infra][WIP]: Show new dep...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5093#issuecomment-86330914 [Test build #29207 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29207/consoleFull) for PR 5093 at commit [`2bc7b27`](https://gith

[GitHub] spark pull request: [SPARK-4123][Project Infra][WIP]: Show new dep...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5093#issuecomment-86330913 [Test build #29207 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29207/consoleFull) for PR 5093 at commit [`2bc7b27`](https://githu

[GitHub] spark pull request: [SPARK-6546][Build] Using the wrong code that ...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5198#issuecomment-86330875 [Test build #29206 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29206/consoleFull) for PR 5198 at commit [`6e0140d`](https://githu

[GitHub] spark pull request: [SPARK-6546][Build] Using the wrong code that ...

2015-03-25 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/5198#issuecomment-86330833 @JoshRosen It should be #4289. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-6546][Build] Using the wrong code that ...

2015-03-25 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/5198#issuecomment-86330726 Do you know which PR broke 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 no

[GitHub] spark pull request: [SPARK-6546][Build] Using the wrong code that ...

2015-03-25 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/5198#issuecomment-86330506 Jenkins, this is ok to test. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not ha

[GitHub] spark pull request: [SPARK-6443][WIP]Could not submit app in stand...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5116#issuecomment-86329688 [Test build #29205 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29205/consoleFull) for PR 5116 at commit [`7a881b3`](https://githu

[GitHub] spark pull request: [SPARK-6198][SQL] Support "select current_data...

2015-03-25 Thread DoingDone9
Github user DoingDone9 commented on the pull request: https://github.com/apache/spark/pull/4995#issuecomment-86329564 anyone will test it ? @marmbrus @srowen --- 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-6350][Mesos] Make mesosExecutorCores co...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5063#issuecomment-86327593 [Test build #29204 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29204/consoleFull) for PR 5063 at commit [`49811dd`](https://githu

[GitHub] spark pull request: [SPARK-6145][SQL] fix ORDER BY on nested field...

2015-03-25 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/5189#issuecomment-86327485 Since we decide to resolve `Sort` in `ResolveSortReferences`, then we need to stop resolve `Sort` in `ResolveReferences`. For example, test("quick") {

[GitHub] spark pull request: [SPARK-6350][Mesos] Make mesosExecutorCores co...

2015-03-25 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/spark/pull/5063#issuecomment-86327232 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

[GitHub] spark pull request: [SPARK-6350][Mesos] Make mesosExecutorCores co...

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

[GitHub] spark pull request: [SPARK-6350][Mesos] Make mesosExecutorCores co...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5063#issuecomment-86326197 [Test build #29202 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29202/consoleFull) for PR 5063 at commit [`49811dd`](https://gith

[GitHub] spark pull request: [SPARK-6199] [SQL] Support CTE in HiveContext ...

2015-03-25 Thread haiyangsea
Github user haiyangsea commented on a diff in the pull request: https://github.com/apache/spark/pull/4929#discussion_r27186638 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -170,21 +170,35 @@ class Analyzer(catalog: Catalog,

[GitHub] spark pull request: [SPARK-5885][MLLIB] Add VectorAssembler as a f...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5196#issuecomment-86324798 [Test build #29199 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29199/consoleFull) for PR 5196 at commit [`29fb6ac`](https://gith

[GitHub] spark pull request: [SPARK-5885][MLLIB] Add VectorAssembler as a f...

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

[GitHub] spark pull request: [SPARK-6546][Build] Using the wrong code that ...

2015-03-25 Thread DoingDone9
Github user DoingDone9 commented on the pull request: https://github.com/apache/spark/pull/5198#issuecomment-86324403 i think this is pressing @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 do

[GitHub] spark pull request: [SPARK-6463][SQL] AttributeSet.equal should co...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5194#issuecomment-86322784 [Test build #29196 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29196/consoleFull) for PR 5194 at commit [`5ed4615`](https://gith

[GitHub] spark pull request: [SPARK-6463][SQL] AttributeSet.equal should co...

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

[GitHub] spark pull request: [Minor][SQL] Minor changes

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5199#issuecomment-86321699 [Test build #29203 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29203/consoleFull) for PR 5199 at commit [`84d15d5`](https://githu

[GitHub] spark pull request: [Minor][SQL] Minor changes

2015-03-25 Thread chenghao-intel
GitHub user chenghao-intel opened a pull request: https://github.com/apache/spark/pull/5199 [Minor][SQL] Minor changes Couple of minor changes for SQL. You can merge this pull request into a Git repository by running: $ git pull https://github.com/chenghao-intel/spark tiny_code

[GitHub] spark pull request: A little spell wrong, but this will make spark...

2015-03-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5198#issuecomment-86320039 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 pro

[GitHub] spark pull request: A little spell wrong, but this will make spark...

2015-03-25 Thread DoingDone9
GitHub user DoingDone9 opened a pull request: https://github.com/apache/spark/pull/5198 A little spell wrong, but this will make spark compile failed!! wrong code : val tmpDir = Files.createTempDir() not Files should File You can merge this pull request into a Git repository by

[GitHub] spark pull request: [SPARK-5654] Integrate SparkR

2015-03-25 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/5096#issuecomment-86315964 We will do some refactor in SerDe, plan to add tests later. This PR is already huge, not convenient to review, I'd like to avoid large changes in this one. --- If your p

[GitHub] spark pull request: [SPARK-6396][Core] Add broadcast timeout

2015-03-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5197#issuecomment-86315457 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 pro

[GitHub] spark pull request: [SPARK-6179][SQL] Add token for "SHOW PRINCIPA...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4902#issuecomment-86314464 [Test build #29201 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29201/consoleFull) for PR 4902 at commit [`0c7550a`](https://gith

[GitHub] spark pull request: [SPARK-6179][SQL] Add token for "SHOW PRINCIPA...

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

[GitHub] spark pull request: [SPARK-6396][Core] Add broadcast timeout

2015-03-25 Thread chidouhu
GitHub user chidouhu opened a pull request: https://github.com/apache/spark/pull/5197 [SPARK-6396][Core] Add broadcast timeout TorrentBroadcast uses fetchBlockSync method of BlockTransferService.scala which wait Duration.Inf. A timeout would be better. You can merge this pull reque

[GitHub] spark pull request: [SPARK-6396][Core] Add broadcast timeout

2015-03-25 Thread chidouhu
Github user chidouhu commented on the pull request: https://github.com/apache/spark/pull/5186#issuecomment-86313560 ok, i will take care of it. 2015-03-25 21:52 GMT+08:00 Sean Owen : > @chidouhu you do not need to delete a PR > each ti

[GitHub] spark pull request: [SPARK-6542][SQL] add CreateStruct

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

[GitHub] spark pull request: [SPARK-6542][SQL] add CreateStruct

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5195#issuecomment-86309631 [Test build #29197 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29197/consoleFull) for PR 5195 at commit [`85f3106`](https://gith

[GitHub] spark pull request: [SPARK-6350][Mesos] Make mesosExecutorCores co...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5063#issuecomment-86309543 [Test build #29202 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29202/consoleFull) for PR 5063 at commit [`49811dd`](https://githu

[GitHub] spark pull request: [SPARK-6179][SQL] Add token for "SHOW PRINCIPA...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4902#issuecomment-86307182 [Test build #29201 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29201/consoleFull) for PR 4902 at commit [`0c7550a`](https://githu

[GitHub] spark pull request: [SPARK-6179][SQL] Add token for "SHOW PRINCIPA...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4902#issuecomment-86305684 [Test build #29200 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29200/consoleFull) for PR 4902 at commit [`c8aeb1c`](https://gith

[GitHub] spark pull request: [SPARK-6179][SQL] Add token for "SHOW PRINCIPA...

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

[GitHub] spark pull request: [SPARK-6483][SQL]Improve ScalaUdf called perfo...

2015-03-25 Thread zzcclp
Github user zzcclp commented on the pull request: https://github.com/apache/spark/pull/5154#issuecomment-86305650 @liancheng , I already set my real name on my GitHub *Name* and JIRA *Full Name*. :smiley: --- If your project is set up for it, you can reply to this email and have you

[GitHub] spark pull request: [SPARK-5885][MLLIB] Add VectorAssembler as a f...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5196#issuecomment-86305463 [Test build #29199 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29199/consoleFull) for PR 5196 at commit [`29fb6ac`](https://githu

[GitHub] spark pull request: [SPARK-6179][SQL] Add token for "SHOW PRINCIPA...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4902#issuecomment-86305065 [Test build #29200 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29200/consoleFull) for PR 4902 at commit [`c8aeb1c`](https://githu

[GitHub] spark pull request: [SPARK-6179][SQL] Add token for "SHOW PRINCIPA...

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

[GitHub] spark pull request: [SPARK-6179][SQL] Add token for "SHOW PRINCIPA...

2015-03-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4902#issuecomment-86303487 [Test build #29198 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29198/consoleFull) for PR 4902 at commit [`ec8bc94`](https://gith

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-03-25 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/3558#discussion_r27184692 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/CatalystConf.scala --- @@ -0,0 +1,53 @@ +/* + * Licensed to the Apache Software F

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-03-25 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/3558#discussion_r27184655 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/CatalystConf.scala --- @@ -0,0 +1,53 @@ +/* + * Licensed to the Apache Software F

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-03-25 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/3558#discussion_r27184671 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -39,11 +40,17 @@ object SimpleAnalyzer extends Analyzer(E

  1   2   3   4   5   >