Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8889
---
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 user yhuai commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143572620
LGTM. Merging to master and branch 1.5.
---
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 user yhuai commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143511190
test 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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143515150
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143515129
[Test build #43053 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43053/console)
for PR 8889 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143515149
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143511258
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143511251
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143511588
[Test build #43053 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43053/consoleFull)
for PR 8889 at commit
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/8889#discussion_r40476685
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -553,60 +553,28 @@ private[hive] class
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/8889#discussion_r40476708
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -553,60 +553,28 @@ private[hive] class
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/8889#discussion_r40477126
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
---
@@ -1223,4 +1223,26 @@ class SQLQuerySuite extends QueryTest
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143358045
LGTM except for a few minor issues.
---
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 user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/8889#discussion_r40476942
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -553,60 +553,28 @@ private[hive] class
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143380515
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143379490
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143381186
[Test build #43036 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43036/consoleFull)
for PR 8889 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143383583
[Test build #43036 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43036/console)
for PR 8889 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143383598
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143383597
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-142837430
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-142837429
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-142837379
[Test build #42962 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42962/console)
for PR 8889 at commit
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8889#discussion_r40339049
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/LogicalRelation.scala
---
@@ -17,23 +17,44 @@
package
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8889#discussion_r40342337
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/LogicalRelation.scala
---
@@ -17,23 +17,44 @@
package
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8889#discussion_r40338157
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/LogicalRelation.scala
---
@@ -17,23 +17,44 @@
package
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8889#discussion_r40340163
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/LogicalRelation.scala
---
@@ -17,23 +17,44 @@
package
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8889#discussion_r40340100
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/LogicalRelation.scala
---
@@ -17,23 +17,44 @@
package
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8889#discussion_r40342268
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
---
@@ -1223,4 +1223,26 @@ class SQLQuerySuite extends QueryTest
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-142996116
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-142996070
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 user yhuai commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143027674
Seems it failed some tests.
---
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143022477
[Test build #42975 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42975/console)
for PR 8889 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143022552
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143022548
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143063695
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143063721
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143064844
[Test build #42986 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42986/consoleFull)
for PR 8889 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143086881
[Test build #42986 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42986/console)
for PR 8889 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143086951
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143086950
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
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-143112924
@liancheng can you also take a look?
---
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-142997788
[Test build #42975 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42975/consoleFull)
for PR 8889 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-142757717
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 user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8889#discussion_r40273156
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/LogicalRelation.scala
---
@@ -17,23 +17,35 @@
package
GitHub user cloud-fan opened a pull request:
https://github.com/apache/spark/pull/8889
[SPARK-10741][SQL] Hive Query Having/OrderBy against Parquet table is not
working
https://issues.apache.org/jira/browse/SPARK-10741
I choose the second approach: do not change output exprIds
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-142757727
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-142758645
[Test build #42934 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42934/consoleFull)
for PR 8889 at commit
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-142758762
cc @yhuai
---
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 user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/8889#discussion_r40273324
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/LogicalRelation.scala
---
@@ -17,23 +17,35 @@
package
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8889#discussion_r40273351
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/LogicalRelation.scala
---
@@ -17,23 +17,35 @@
package
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8889#discussion_r40274896
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/LogicalRelation.scala
---
@@ -17,23 +17,35 @@
package
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-142774336
cc @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 this
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-142774667
[Test build #42934 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42934/console)
for PR 8889 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-142774718
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-142774719
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-142774597
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-142774609
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-142775374
[Test build #42945 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42945/consoleFull)
for PR 8889 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-142775586
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-142775581
[Test build #42945 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42945/console)
for PR 8889 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-142775585
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-142814881
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-142814865
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8889#issuecomment-142815578
[Test build #42962 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42962/consoleFull)
for PR 8889 at commit
65 matches
Mail list logo
