[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 [[Br

[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 <: Ne

[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 [[Br

[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 PartitioningCollectio

[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 OrderedDistribution(ord

[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 RoundRobinPartitionin

[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 [`d42d97d`](https://

[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): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[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 [`79314b5`](https://gi

[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 [`f2e6a25`](https://gi

[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 [`c9bd987`](https://gi

[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 [`3f2980a`](https://gi

[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 [`ff9900b`](https://gi

[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 OrderedDistribution(ord

[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 [`e21fded`](https://gi

[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 [`de512f6`](https://gi

[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 [`4783c4c`](https://g

[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 projec

[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): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[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 [`4783c4c`](https://gi

[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 [`9f393f0`](https://

[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 [`d42d97d`](https://g

[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 [`9f393f0`](https://g

[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 h

[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 enabl

[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 a

<    1   2   3   4   5