[GitHub] spark pull request: [SPARK-13136][SQL] Create a dedicated Broadcas...

2016-02-21 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/11083#discussion_r53563531 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/exchange.scala --- @@ -261,10 +266,71 @@ object Exchange { } /** + * A

[GitHub] spark pull request: [SPARK-13416][GraphX] Add positive check for o...

2016-02-21 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11284#issuecomment-186777256 Thanks - I've merged this in master. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[GitHub] spark pull request: [SPARK-13186][Streaming]migrate away from Sync...

2016-02-21 Thread huaxingao
Github user huaxingao commented on a diff in the pull request: https://github.com/apache/spark/pull/11250#discussion_r53563513 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/dstream/FileInputDStream.scala --- @@ -163,8 +166,11 @@ class FileInputDStream[K, V, F <:

[GitHub] spark pull request: [SPARK-13136][SQL] Create a dedicated Broadcas...

2016-02-21 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/11083#discussion_r53563517 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/exchange.scala --- @@ -261,10 +266,71 @@ object Exchange { } /** + * A

[GitHub] spark pull request: [SPARK-13136][SQL] Create a dedicated Broadcas...

2016-02-21 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/11083#discussion_r53563485 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/physical/partitioning.scala --- @@ -351,3 +376,21 @@ case class

[GitHub] spark pull request: [SPARK-13136][SQL] Create a dedicated Broadcas...

2016-02-21 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/11083#discussion_r53563479 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/physical/partitioning.scala --- @@ -76,6 +77,27 @@ case class

[GitHub] spark pull request: [SPARK-13136][SQL] Create a dedicated Broadcas...

2016-02-21 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/11083#discussion_r53563471 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/physical/partitioning.scala --- @@ -213,7 +235,10 @@ case class

[GitHub] spark pull request: [SPARK-13416][GraphX] Add positive check for o...

2016-02-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11284#issuecomment-186775975 **[Test build #2557 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2557/consoleFull)** for PR 11284 at commit

[GitHub] spark pull request: [SPARK-12799] Simplify various string output f...

2016-02-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10757#issuecomment-186775925 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

[GitHub] spark pull request: [SPARK-12799] Simplify various string output f...

2016-02-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10757#issuecomment-186775926 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13420][SQL] Rename Subquery logical pla...

2016-02-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11288#issuecomment-186775746 **[Test build #51618 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51618/consoleFull)** for PR 11288 at commit

[GitHub] spark pull request: [SPARK-13398][STREAMING] Change FileWriteAhead...

2016-02-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11287#issuecomment-186775743 **[Test build #51616 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51616/consoleFull)** for PR 11287 at commit

[GitHub] spark pull request: [SPARK-13306][SQL] Addendum to uncorrelated sc...

2016-02-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11285#issuecomment-186775782 **[Test build #51611 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51611/consoleFull)** for PR 11285 at commit

[GitHub] spark pull request: [SPARK-13399][STREAMING] Fix checkpointsuite t...

2016-02-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11286#issuecomment-186775751 **[Test build #51613 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51613/consoleFull)** for PR 11286 at commit

[GitHub] spark pull request: [SPARK-13017][Docs] Replace example code in ml...

2016-02-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11142#issuecomment-186775641 **[Test build #51612 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51612/consoleFull)** for PR 11142 at commit

[GitHub] spark pull request: [SPARK-13136][SQL] Create a dedicated Broadcas...

2016-02-21 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/11083#discussion_r53563435 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/physical/partitioning.scala --- @@ -76,6 +77,27 @@ case class

[GitHub] spark pull request: [SPARK-12799] Simplify various string output f...

2016-02-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10757#issuecomment-186775478 **[Test build #51619 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51619/consoleFull)** for PR 10757 at commit

[GitHub] spark pull request: [SPARK-13306][SQL] Addendum to uncorrelated sc...

2016-02-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11285#issuecomment-186775454 **[Test build #51614 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51614/consoleFull)** for PR 11285 at commit

[GitHub] spark pull request: [SPARK-13233][SQL][WIP] Python Dataset

2016-02-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7#issuecomment-186775386 **[Test build #51617 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51617/consoleFull)** for PR 7 at commit

[GitHub] spark pull request: [SPARK-13233][SQL][WIP] Python Dataset

2016-02-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7#issuecomment-186775387 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-13233][SQL][WIP] Python Dataset

2016-02-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7#issuecomment-186775388 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13233][SQL][WIP] Python Dataset

2016-02-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7#issuecomment-186775311 **[Test build #51617 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51617/consoleFull)** for PR 7 at commit

[GitHub] spark pull request: [SPARK-13410][SQL] Support unionAll for DataFr...

2016-02-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11279#issuecomment-186775275 **[Test build #2556 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2556/consoleFull)** for PR 11279 at commit

[GitHub] spark pull request: [SPARK-13416][GraphX] Add positive check for o...

2016-02-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11284#issuecomment-186775224 **[Test build #2557 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2557/consoleFull)** for PR 11284 at commit

[GitHub] spark pull request: [SPARK-13410][SQL] Support unionAll for DataFr...

2016-02-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11279#issuecomment-186775231 **[Test build #2556 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2556/consoleFull)** for PR 11279 at commit

[GitHub] spark pull request: [SPARK-12746][ML] ArrayType(_, true) should al...

2016-02-21 Thread Earthson
Github user Earthson commented on the pull request: https://github.com/apache/spark/pull/11237#issuecomment-186774933 @srowen Is this ok to merge? --- 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-13233][SQL][WIP] Python Dataset

2016-02-21 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/7#issuecomment-186774385 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

[GitHub] spark pull request: [SPARK-13420][SQL] Rename Subquery logical pla...

2016-02-21 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11288#issuecomment-186773764 cc @davies --- 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: [SPARK-13420][SQL] Rename Subquery logical pla...

2016-02-21 Thread rxin
GitHub user rxin opened a pull request: https://github.com/apache/spark/pull/11288 [SPARK-13420][SQL] Rename Subquery logical plan to SubqueryAlias ## What changes were proposed in this pull request? This patch renames logical.Subquery to logical.SubqueryAlias, which is a more

<    1   2   3   4   5