[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

2015-11-06 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/5919#issuecomment-154612297 @rxin @marmbrus This fix is unable to resolve the condition ambiguity for nested self join. I also found the self joins could generate incorrect results. --- If

[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

2015-05-05 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/5919#issuecomment-99288746 ... and branch 1.4 --- 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-6231][SQL/DF] Automatically resolve joi...

2015-05-05 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/5919#issuecomment-99287769 Thanks! Merged to 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

[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

2015-05-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/5919 --- 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-6231][SQL/DF] Automatically resolve joi...

2015-05-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5919#issuecomment-99256110 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 project

[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

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

[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

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

[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

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

[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

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

[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

2015-05-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5919#issuecomment-99269530 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 project

[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

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

[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

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

[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

2015-05-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5919#issuecomment-99252464 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 project

[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

2015-05-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5919#issuecomment-99248637 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 project

[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

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

[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

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

[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

2015-05-05 Thread rxin
Github user rxin closed the pull request at: https://github.com/apache/spark/pull/5919 --- 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-6231][SQL/DF] Automatically resolve joi...

2015-05-05 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/5919#issuecomment-99249025 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-6231][SQL/DF] Automatically resolve joi...

2015-05-05 Thread rxin
GitHub user rxin reopened a pull request: https://github.com/apache/spark/pull/5919 [SPARK-6231][SQL/DF] Automatically resolve join condition ambiguity for self-joins. See the comment in join function for more information. You can merge this pull request into a Git repository by

[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

2015-05-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5919#issuecomment-99249768 Merged build started. --- 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-6231][SQL/DF] Automatically resolve joi...

2015-05-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5919#issuecomment-99249749 Merged build triggered. --- 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-6231][SQL/DF] Automatically resolve joi...

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

[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

2015-05-05 Thread rxin
GitHub user rxin opened a pull request: https://github.com/apache/spark/pull/5919 [SPARK-6231][SQL/DF] Automatically resolve join condition ambiguity for self-joins. You can merge this pull request into a Git repository by running: $ git pull https://github.com/rxin/spark

[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

2015-05-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5919#issuecomment-99218375 Merged build triggered. --- 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-6231][SQL/DF] Automatically resolve joi...

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

[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

2015-05-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5919#issuecomment-99218463 Merged build started. --- 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-6231][SQL/DF] Automatically resolve joi...

2015-05-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5919#issuecomment-99220696 Merged build triggered. --- 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-6231][SQL/DF] Automatically resolve joi...

2015-05-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5919#issuecomment-99220717 Merged build started. --- 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-6231][SQL/DF] Automatically resolve joi...

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

[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

2015-05-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5919#issuecomment-99225205 Merged build triggered. --- 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-6231][SQL/DF] Automatically resolve joi...

2015-05-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5919#issuecomment-99225318 Merged build started. --- 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-6231][SQL/DF] Automatically resolve joi...

2015-05-05 Thread damiencarol
Github user damiencarol commented on a diff in the pull request: https://github.com/apache/spark/pull/5919#discussion_r29712214 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/namedExpressions.scala --- @@ -171,6 +171,11 @@ case class

[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

2015-05-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5919#discussion_r29712513 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/namedExpressions.scala --- @@ -171,6 +171,11 @@ case class AttributeReference(

[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

2015-05-05 Thread damiencarol
Github user damiencarol commented on a diff in the pull request: https://github.com/apache/spark/pull/5919#discussion_r29713157 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/namedExpressions.scala --- @@ -171,6 +171,11 @@ case class

[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

2015-05-05 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/5919#discussion_r29713542 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/namedExpressions.scala --- @@ -171,6 +171,11 @@ case class

[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

2015-05-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5919#discussion_r29713770 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/namedExpressions.scala --- @@ -171,6 +171,11 @@ case class AttributeReference(

[GitHub] spark pull request: [SPARK-6231][SQL/DF] Automatically resolve joi...

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