Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/13121
---
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 rxin commented on the pull request:
https://github.com/apache/spark/pull/13121#issuecomment-220811816
Merging in master/2.0. Thanks.
---
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/13121#issuecomment-220806369
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/13121#issuecomment-220806368
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13121#issuecomment-220806334
**[Test build #59083 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59083/consoleFull)**
for PR 13121 at commit
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/13121#issuecomment-220804464
LGTM pending 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 this
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13121#issuecomment-220803288
**[Test build #59083 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59083/consoleFull)**
for PR 13121 at commit
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/13121#issuecomment-220803118
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13121#issuecomment-220801790
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/13121#issuecomment-220801786
**[Test build #59080 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59080/consoleFull)**
for PR 13121 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13121#issuecomment-220801791
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/13121#issuecomment-220801477
**[Test build #59080 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59080/consoleFull)**
for PR 13121 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/13121#discussion_r64140446
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/internal/SQLConfSuite.scala ---
@@ -107,6 +107,53 @@ class SQLConfSuite extends QueryTest with
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/13121#issuecomment-220795764
LGTM
---
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 cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13121#discussion_r64138917
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/internal/SQLConfSuite.scala ---
@@ -107,6 +107,53 @@ class SQLConfSuite extends QueryTest with
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13121#issuecomment-220764740
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/13121#issuecomment-220764742
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/13121#issuecomment-220764703
**[Test build #59067 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59067/consoleFull)**
for PR 13121 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13121#issuecomment-220762164
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/13121#issuecomment-220762158
**[Test build #59065 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59065/consoleFull)**
for PR 13121 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13121#issuecomment-220762165
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/13121#issuecomment-220762068
**[Test build #59067 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59067/consoleFull)**
for PR 13121 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13121#issuecomment-220761910
**[Test build #59065 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59065/consoleFull)**
for PR 13121 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/13121#discussion_r64127289
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -778,5 +778,11 @@ private[sql] class SQLConf extends Serializable
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/13121#discussion_r64126788
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -778,5 +778,11 @@ private[sql] class SQLConf extends Serializable with
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13121#issuecomment-220753740
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/13121#issuecomment-220753741
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/13121#issuecomment-220753705
**[Test build #59040 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59040/consoleFull)**
for PR 13121 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13121#issuecomment-220736213
**[Test build #59040 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59040/consoleFull)**
for PR 13121 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/13121#discussion_r64095187
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -778,5 +778,11 @@ private[sql] class SQLConf extends Serializable
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/13121#discussion_r64090794
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -778,5 +778,11 @@ private[sql] class SQLConf extends Serializable
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/13121#discussion_r64069854
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -778,5 +778,11 @@ private[sql] class SQLConf extends Serializable with
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/13121#discussion_r64069789
--- Diff:
sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/SparkSQLCLIDriver.scala
---
@@ -311,77 +311,78 @@ private[hive] class
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/13121#issuecomment-220637177
cc @rxin @hvanhovell @cloud-fan Could you also review this command? Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/13121#issuecomment-220177169
@hvanhovell Does the latest change look fine to you? Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13121#issuecomment-219628299
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/13121#issuecomment-219628296
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13121#issuecomment-219628173
**[Test build #58664 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58664/consoleFull)**
for PR 13121 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13121#issuecomment-219618421
**[Test build #58664 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58664/consoleFull)**
for PR 13121 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/13121#discussion_r63416697
--- Diff:
sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/SparkSQLCLIDriver.scala
---
@@ -312,7 +312,7 @@ private[hive]
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/13121#discussion_r63416647
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -778,5 +778,11 @@ private[sql] class SQLConf extends Serializable
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/13121#discussion_r63416626
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/SetCommand.scala
---
@@ -118,3 +116,19 @@ case class SetCommand(kv:
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/13121#discussion_r63415729
--- Diff:
sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/SparkSQLCLIDriver.scala
---
@@ -312,7 +312,7 @@ private[hive]
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/13121#discussion_r63415650
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -778,5 +778,11 @@ private[sql] class SQLConf extends Serializable
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/13121#discussion_r63415352
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/SetCommand.scala
---
@@ -118,3 +116,19 @@ case class SetCommand(kv:
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13121#issuecomment-219265969
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/13121#issuecomment-219265968
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13121#issuecomment-219265948
**[Test build #58609 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58609/consoleFull)**
for PR 13121 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13121#issuecomment-219263713
**[Test build #58609 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58609/consoleFull)**
for PR 13121 at commit
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/13121
[SPARK-15330] [SQL] Implement Reset Command
What changes were proposed in this pull request?
Like `Set` Command in Hive, `Reset` is also supported by Hive. See the
link:
50 matches
Mail list logo