[GitHub] spark pull request: [SPARK-14007] [SQL] Manage the memory used by ...

2016-03-21 Thread tedyu
Github user tedyu commented on a diff in the pull request: https://github.com/apache/spark/pull/11826#discussion_r56918584 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/joins/ShuffledHashJoin.scala --- @@ -55,11 +56,45 @@ case class ShuffledHashJoin( over

[GitHub] spark pull request: [SPARK-14007] [SQL] Manage the memory used by ...

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

[GitHub] spark pull request: [SPARK-14007] [SQL] Manage the memory used by ...

2016-03-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11826#issuecomment-199498430 Test PASSed. 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-14007] [SQL] Manage the memory used by ...

2016-03-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11826#issuecomment-199498168 **[Test build #53695 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53695/consoleFull)** for PR 11826 at commit [`6a99c42`](https://g

[GitHub] spark pull request: [SPARK-14007] [SQL] Manage the memory used by ...

2016-03-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/11826 --- 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 ena

[GitHub] spark pull request: [SPARK-14007] [SQL] Manage the memory used by ...

2016-03-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11826#issuecomment-199412443 **[Test build #53695 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53695/consoleFull)** for PR 11826 at commit [`6a99c42`](https://gi

[GitHub] spark pull request: [SPARK-14007] [SQL] Manage the memory used by ...

2016-03-21 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/11826#issuecomment-199411859 Merging this into 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 not have t

[GitHub] spark pull request: [SPARK-14007] [SQL] Manage the memory used by ...

2016-03-21 Thread davies
Github user davies commented on a diff in the pull request: https://github.com/apache/spark/pull/11826#discussion_r56869370 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/joins/ShuffledHashJoin.scala --- @@ -55,11 +56,43 @@ case class ShuffledHashJoin( ove

[GitHub] spark pull request: [SPARK-14007] [SQL] Manage the memory used by ...

2016-03-21 Thread sameeragarwal
Github user sameeragarwal commented on the pull request: https://github.com/apache/spark/pull/11826#issuecomment-199162704 LGTM, just one clarifying question. --- 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-14007] [SQL] Manage the memory used by ...

2016-03-21 Thread sameeragarwal
Github user sameeragarwal commented on a diff in the pull request: https://github.com/apache/spark/pull/11826#discussion_r56788993 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/joins/ShuffledHashJoin.scala --- @@ -55,11 +56,43 @@ case class ShuffledHashJoin(

[GitHub] spark pull request: [SPARK-14007] [SQL] Manage the memory used by ...

2016-03-21 Thread sameeragarwal
Github user sameeragarwal commented on a diff in the pull request: https://github.com/apache/spark/pull/11826#discussion_r56788902 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/joins/ShuffledHashJoin.scala --- @@ -55,11 +56,43 @@ case class ShuffledHashJoin(

[GitHub] spark pull request: [SPARK-14007] [SQL] Manage the memory used by ...

2016-03-21 Thread sameeragarwal
Github user sameeragarwal commented on a diff in the pull request: https://github.com/apache/spark/pull/11826#discussion_r56788809 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/joins/ShuffledHashJoin.scala --- @@ -55,11 +56,43 @@ case class ShuffledHashJoin(

[GitHub] spark pull request: [SPARK-14007] [SQL] Manage the memory used by ...

2016-03-21 Thread sameeragarwal
Github user sameeragarwal commented on a diff in the pull request: https://github.com/apache/spark/pull/11826#discussion_r56788831 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/joins/ShuffledHashJoin.scala --- @@ -55,11 +56,43 @@ case class ShuffledHashJoin(

[GitHub] spark pull request: [SPARK-14007] [SQL] Manage the memory used by ...

2016-03-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11826#issuecomment-198550344 **[Test build #2652 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2652/consoleFull)** for PR 11826 at commit [`6382eea`](https://g

[GitHub] spark pull request: [SPARK-14007] [SQL] Manage the memory used by ...

2016-03-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11826#issuecomment-198475914 **[Test build #53547 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53547/consoleFull)** for PR 11826 at commit [`6382eea`](https://gi

[GitHub] spark pull request: [SPARK-14007] [SQL] Manage the memory used by ...

2016-03-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11826#issuecomment-198481175 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-14007] [SQL] Manage the memory used by ...

2016-03-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11826#issuecomment-198582840 **[Test build #2652 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2652/consoleFull)** for PR 11826 at commit [`6382eea`](https://

[GitHub] spark pull request: [SPARK-14007] [SQL] Manage the memory used by ...

2016-03-18 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11826#issuecomment-198622234 cc @sameeragarwal for review --- 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-14007] [SQL] Manage the memory used by ...

2016-03-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11826#issuecomment-198481171 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-14007] [SQL] Manage the memory used by ...

2016-03-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11826#issuecomment-198481145 **[Test build #53547 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53547/consoleFull)** for PR 11826 at commit [`6382eea`](https://g

[GitHub] spark pull request: [SPARK-14007] [SQL] Manage the memory used by ...

2016-03-18 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/11826#issuecomment-198587100 cc @rxin --- 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 enab