Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13386
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 not have this feature
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13386
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59720/
Test
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13386
**[Test build #59720 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59720/consoleFull)**
for PR 13386 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13386
**[Test build #59720 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59720/consoleFull)**
for PR 13386 at commit
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/13386#discussion_r65306005
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -936,7 +936,39 @@ class SparkSqlAstBuilder(conf: SQLConf) extends
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/13386#discussion_r65305666
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -936,7 +936,39 @@ class SparkSqlAstBuilder(conf: SQLConf) extends
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/13386#discussion_r65305383
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveSessionState.scala ---
@@ -139,22 +139,6 @@ private[hive] class HiveSessionState(sparkSession:
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/13386#discussion_r65305344
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -447,52 +447,20 @@ private[hive] class
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/13386
LGTM, minor comments only.
---
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 andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13386#discussion_r65266137
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveSessionState.scala ---
@@ -139,22 +139,6 @@ private[hive] class
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13386#discussion_r65265900
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -936,7 +936,39 @@ class SparkSqlAstBuilder(conf: SQLConf)
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/13386#discussion_r65265712
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -936,7 +936,39 @@ class SparkSqlAstBuilder(conf: SQLConf) extends
Github user ericl commented on the pull request:
https://github.com/apache/spark/pull/13386
looks good, just have a couple questions
---
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 ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/13386#discussion_r65265388
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -447,52 +447,20 @@ private[hive] class
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/13386#discussion_r65265253
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -936,7 +936,39 @@ class SparkSqlAstBuilder(conf: SQLConf) extends
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/13386#discussion_r65264707
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -936,7 +936,39 @@ class SparkSqlAstBuilder(conf: SQLConf) extends
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13386
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59660/
Test
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13386
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 does not have this
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13386
**[Test build #59660 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59660/consoleFull)**
for PR 13386 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13386
**[Test build #59660 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59660/consoleFull)**
for PR 13386 at commit
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/13386
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 feature
enabled and wishes
21 matches
Mail list logo