Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8990
---
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 enab
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146665924
Merging 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 this featu
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146663563
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146663391
[Test build #43405 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43405/console)
for PR 8990 at commit
[`d3eed7b`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146663562
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146632471
[Test build #43405 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43405/consoleFull)
for PR 8990 at commit
[`d3eed7b`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146630755
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146630776
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 user yhuai commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146627504
LGTM. Just three minor comments.
---
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 user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41540476
--- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala ---
@@ -77,6 +77,16 @@ private[hive] case class CreateTableAsSelect(
childr
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41540520
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/CreateViewAsSelect.scala
---
@@ -0,0 +1,95 @@
+/*
+ * Licensed to the Apache Soft
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41540357
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -588,6 +589,27 @@ private[hive] class HiveMetastoreCatalog(val
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41540255
--- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala ---
@@ -77,6 +77,16 @@ private[hive] case class CreateTableAsSelect(
childrenRe
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41540142
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -588,6 +589,27 @@ private[hive] class HiveMetastoreCatalog(val cli
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146510565
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146510564
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146510450
[Test build #43392 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43392/console)
for PR 8990 at commit
[`a2a207b`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146473488
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146473487
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146472078
[Test build #43392 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43392/consoleFull)
for PR 8990 at commit
[`a2a207b`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146470783
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146470805
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146468890
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146468910
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146438011
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146438007
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146437957
[Test build #43381 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43381/console)
for PR 8990 at commit
[`98fb88c`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146421211
[Test build #43381 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43381/consoleFull)
for PR 8990 at commit
[`98fb88c`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146420903
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146420914
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146399845
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146399756
[Test build #43370 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43370/console)
for PR 8990 at commit
[`44fcbf9`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146385709
[Test build #43370 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43370/consoleFull)
for PR 8990 at commit
[`44fcbf9`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146385387
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146385401
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146384271
[Test build #43368 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43368/console)
for PR 8990 at commit
[`c7afcd4`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146384278
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146384275
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146383414
[Test build #43368 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43368/consoleFull)
for PR 8990 at commit
[`c7afcd4`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146381878
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146381892
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 user yhuai commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146334125
Let's also take a look at `ALTER VIEW AS SELECT` and `CREATE OR REPLACE
VIEW`.
---
If your project is set up for it, you can reply to this email and have your
reply appea
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146333801
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146333741
[Test build #43347 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43347/console)
for PR 8990 at commit
[`58fac13`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146333798
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146330011
[Test build #43347 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43347/consoleFull)
for PR 8990 at commit
[`58fac13`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146328464
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146328373
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146316203
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146316207
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146316123
[Test build #43334 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43334/console)
for PR 8990 at commit
[`961c4a2`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146314841
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146314778
[Test build #43332 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43332/console)
for PR 8990 at commit
[`d0a5d3e`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146314840
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146292173
[Test build #43334 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43334/consoleFull)
for PR 8990 at commit
[`961c4a2`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146291102
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146291123
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146285939
[Test build #43332 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43332/consoleFull)
for PR 8990 at commit
[`d0a5d3e`](https://gith
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146285841
overall this LGTM, thanks for working on it.
---
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 pro
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41427063
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
---
@@ -1248,4 +1248,47 @@ class SQLQuerySuite extends QueryTest w
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41426990
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/CreateViewAsSelect.scala
---
@@ -0,0 +1,88 @@
+/*
+ * Licensed to the Apache S
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41426940
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/ClientWrapper.scala ---
@@ -354,6 +354,28 @@ private[hive] class ClientWrapper(
q
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41426883
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/ClientInterface.scala
---
@@ -45,6 +45,7 @@ private[hive] case class HivePartition(
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146284281
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146284304
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 user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41425875
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
---
@@ -1248,4 +1248,47 @@ class SQLQuerySuite extends QueryTest
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41418669
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
---
@@ -1248,4 +1248,47 @@ class SQLQuerySuite extends QueryTest with
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41418218
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/CreateViewAsSelect.scala
---
@@ -0,0 +1,77 @@
+/*
+ * Licensed to the Apache Soft
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41417693
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/ClientWrapper.scala ---
@@ -354,6 +354,25 @@ private[hive] class ClientWrapper(
qlTa
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41417537
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/CreateViewAsSelect.scala
---
@@ -0,0 +1,77 @@
+/*
+ * Licensed to the Apache Soft
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41417365
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/CreateViewAsSelect.scala
---
@@ -0,0 +1,77 @@
+/*
+ * Licensed to the Apache Soft
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41416845
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/CreateViewAsSelect.scala
---
@@ -0,0 +1,77 @@
+/*
+ * Licensed to the Apache Soft
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41416396
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/CreateViewAsSelect.scala
---
@@ -0,0 +1,77 @@
+/*
+ * Licensed to the Apache Soft
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41415507
--- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala ---
@@ -342,7 +358,8 @@ private[hive] object HiveQl extends Logging {
}
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146067832
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146067831
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146067781
[Test build #43310 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43310/console)
for PR 8990 at commit
[`c388717`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146048553
[Test build #43310 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43310/consoleFull)
for PR 8990 at commit
[`c388717`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146047882
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146047865
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146037100
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146037099
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146037066
[Test build #43301 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43301/console)
for PR 8990 at commit
[`8e1e4b8`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146014456
[Test build #43301 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43301/consoleFull)
for PR 8990 at commit
[`8e1e4b8`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146012318
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146012307
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146003721
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146003615
[Test build #43294 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43294/console)
for PR 8990 at commit
[`d54ec33`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-146003723
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-145976353
[Test build #43294 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43294/consoleFull)
for PR 8990 at commit
[`d54ec33`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-145975845
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8990#issuecomment-145975883
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 user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41300835
--- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala ---
@@ -563,6 +580,77 @@
https://cwiki.apache.org/confluence/display/Hive/Enhanced+Aggr
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41299272
--- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala ---
@@ -563,6 +580,77 @@
https://cwiki.apache.org/confluence/display/Hive/Enhanced+
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41295949
--- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala ---
@@ -563,6 +580,77 @@
https://cwiki.apache.org/confluence/display/Hive/Enhanced+Aggr
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41295186
--- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala ---
@@ -563,6 +580,77 @@
https://cwiki.apache.org/confluence/display/Hive/Enhanced+
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41225536
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
---
@@ -1248,4 +1248,12 @@ class SQLQuerySuite extends QueryTest with
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41225405
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
---
@@ -1248,4 +1248,12 @@ class SQLQuerySuite extends QueryTest with
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41225256
--- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala ---
@@ -563,6 +580,77 @@
https://cwiki.apache.org/confluence/display/Hive/Enhanced+Aggr
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8990#discussion_r41225160
--- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala ---
@@ -563,6 +580,77 @@
https://cwiki.apache.org/confluence/display/Hive/Enhanced+Aggr
1 - 100 of 115 matches
Mail list logo