Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64926605
--- Diff:
sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/CliSuite.scala
---
@@ -238,4 +238,23 @@ class CliSuite extends
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64441151
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -774,13 +774,42 @@ class SparkSqlAstBuilder(conf: SQLConf)
Github user tedyu commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64405314
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -774,13 +774,42 @@ class SparkSqlAstBuilder(conf: SQLConf)
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64344722
--- Diff:
sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/CliSuite.scala
---
@@ -238,4 +238,23 @@ class CliSuite extends
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64333199
--- Diff:
sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/CliSuite.scala
---
@@ -238,4 +238,23 @@ class CliSuite extends
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64325981
--- Diff:
sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/CliSuite.scala
---
@@ -238,4 +238,23 @@ class CliSuite extends
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64325475
--- Diff:
sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/CliSuite.scala
---
@@ -238,4 +238,23 @@ class CliSuite extends
Github user xwu0226 commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-221135361
@liancheng @cloud-fan @rxin Thank you very much!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/13212
---
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 liancheng commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-221135020
Merging to master and branch-2.0.
---
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/13212#issuecomment-221125197
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/13212#issuecomment-221125196
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/13212#issuecomment-221124995
**[Test build #59154 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59154/consoleFull)**
for PR 13212 at commit
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-221105795
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 xwu0226 commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-221103983
@liancheng Thank you very much for your time!!
---
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 liancheng commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-221102993
LGTM pending Jenkins.
---
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 liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64293071
--- Diff:
sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/SparkSQLCLIDriver.scala
---
@@ -295,9 +294,7 @@ private[hive] class
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-221097798
**[Test build #59154 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59154/consoleFull)**
for PR 13212 at commit
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64281350
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala
---
@@ -46,3 +51,52 @@ case class AddFileCommand(path: String)
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64281099
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -774,13 +774,42 @@ class SparkSqlAstBuilder(conf: SQLConf)
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-221082584
In general looks good except for a few minor comments. Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64279884
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -774,13 +774,42 @@ class SparkSqlAstBuilder(conf: SQLConf)
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64278795
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala
---
@@ -46,3 +51,52 @@ case class AddFileCommand(path: String)
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64278535
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala
---
@@ -46,3 +51,52 @@ case class AddFileCommand(path: String)
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64266371
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -774,13 +774,42 @@ class SparkSqlAstBuilder(conf: SQLConf)
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64264107
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala
---
@@ -46,3 +51,52 @@ case class AddFileCommand(path: String)
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64262960
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala
---
@@ -46,3 +51,52 @@ case class AddFileCommand(path: String)
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64261638
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala
---
@@ -46,3 +51,52 @@ case class AddFileCommand(path: String)
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64261520
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala
---
@@ -46,3 +51,52 @@ case class AddFileCommand(path: String)
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64261332
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala
---
@@ -46,3 +51,52 @@ case class AddFileCommand(path: String)
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64257719
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -774,13 +774,42 @@ class SparkSqlAstBuilder(conf: SQLConf)
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64257331
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala
---
@@ -46,3 +51,52 @@ case class AddFileCommand(path: String)
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64256691
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala
---
@@ -46,3 +51,52 @@ case class AddFileCommand(path: String)
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64256050
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala
---
@@ -46,3 +51,52 @@ case class AddFileCommand(path: String)
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64255661
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -774,13 +774,42 @@ class SparkSqlAstBuilder(conf: SQLConf)
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64255724
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -774,13 +774,42 @@ class SparkSqlAstBuilder(conf: SQLConf)
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64255403
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala
---
@@ -46,3 +51,52 @@ case class AddFileCommand(path: String)
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64255488
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -774,13 +774,42 @@ class SparkSqlAstBuilder(conf: SQLConf)
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64255345
--- Diff:
sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 ---
@@ -117,7 +117,7 @@ statement
tableIdentifier
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64255039
--- Diff: core/src/test/scala/org/apache/spark/SparkContextSuite.scala ---
@@ -156,6 +156,18 @@ class SparkContextSuite extends SparkFunSuite with
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64254938
--- Diff: core/src/test/scala/org/apache/spark/SparkContextSuite.scala ---
@@ -156,6 +156,18 @@ class SparkContextSuite extends SparkFunSuite with
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64253549
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -774,13 +774,42 @@ class SparkSqlAstBuilder(conf: SQLConf)
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64253417
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -774,13 +774,42 @@ class SparkSqlAstBuilder(conf: SQLConf)
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64253346
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -774,13 +774,42 @@ class SparkSqlAstBuilder(conf: SQLConf)
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64252871
--- Diff:
sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 ---
@@ -117,7 +117,7 @@ statement
tableIdentifier
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64252495
--- Diff: core/src/test/scala/org/apache/spark/SparkContextSuite.scala ---
@@ -156,6 +156,18 @@ class SparkContextSuite extends SparkFunSuite with
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64252462
--- Diff: core/src/test/scala/org/apache/spark/SparkContextSuite.scala ---
@@ -156,6 +156,18 @@ class SparkContextSuite extends SparkFunSuite with
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64252524
--- Diff: core/src/test/scala/org/apache/spark/SparkContextSuite.scala ---
@@ -156,6 +156,18 @@ class SparkContextSuite extends SparkFunSuite with
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-220847276
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/13212#issuecomment-220847275
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/13212#issuecomment-220847232
**[Test build #59111 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59111/consoleFull)**
for PR 13212 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-220840805
**[Test build #59111 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59111/consoleFull)**
for PR 13212 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-220820162
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/13212#issuecomment-220820157
**[Test build #59095 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59095/consoleFull)**
for PR 13212 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-220820160
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/13212#issuecomment-220819781
**[Test build #59095 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59095/consoleFull)**
for PR 13212 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-220743518
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/13212#issuecomment-220743520
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/13212#issuecomment-220743390
**[Test build #59033 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59033/consoleFull)**
for PR 13212 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-220724552
**[Test build #59033 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59033/consoleFull)**
for PR 13212 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-220702201
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/13212#issuecomment-220702202
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/13212#issuecomment-220701995
**[Test build #59011 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59011/consoleFull)**
for PR 13212 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-220670523
**[Test build #59011 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59011/consoleFull)**
for PR 13212 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-220668027
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/13212#issuecomment-220668030
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/13212#issuecomment-220668018
**[Test build #59010 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59010/consoleFull)**
for PR 13212 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-220666757
**[Test build #59010 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59010/consoleFull)**
for PR 13212 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-220662246
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/13212#issuecomment-220662248
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/13212#issuecomment-220661923
**[Test build #58998 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58998/consoleFull)**
for PR 13212 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-220629211
**[Test build #58998 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58998/consoleFull)**
for PR 13212 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-220552292
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/13212#issuecomment-220552294
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/13212#issuecomment-220552097
**[Test build #58958 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58958/consoleFull)**
for PR 13212 at commit
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r64000637
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala
---
@@ -46,3 +51,56 @@ case class AddFile(path: String) extends
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r63999888
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala
---
@@ -46,3 +51,56 @@ case class AddFile(path: String) extends
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r63999507
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala
---
@@ -46,3 +51,56 @@ case class AddFile(path: String) extends
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r63999183
--- Diff:
sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/SparkSQLCLIDriver.scala
---
@@ -295,9 +294,7 @@ private[hive] class
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r63999050
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -713,13 +713,40 @@ class SparkSqlAstBuilder(conf: SQLConf)
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r63999023
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala
---
@@ -46,3 +51,56 @@ case class AddFile(path: String) extends
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r63998812
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala
---
@@ -46,3 +51,56 @@ case class AddFile(path: String) extends
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r63998655
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -713,13 +713,40 @@ class SparkSqlAstBuilder(conf: SQLConf)
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-220531077
**[Test build #58958 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58958/consoleFull)**
for PR 13212 at commit
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-220530546
cc @cloud-fan
---
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 the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-220530387
ok to test
---
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 xwu0226 commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-220523283
@rxin Updated the code. Please help double check. Thank you very much!
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r63992232
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -1387,6 +1387,27 @@ class SparkContext(config: SparkConf) extends
Logging with
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r63992144
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -1387,6 +1387,27 @@ class SparkContext(config: SparkConf) extends
Logging with
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r63992055
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -1387,6 +1387,27 @@ class SparkContext(config: SparkConf) extends
Logging with
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r63991418
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala
---
@@ -46,3 +46,33 @@ case class AddFile(path: String) extends
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r63991451
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -1387,6 +1387,27 @@ class SparkContext(config: SparkConf) extends
Logging with
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r63991431
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala
---
@@ -46,3 +46,33 @@ case class AddFile(path: String) extends
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r63991280
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -1387,6 +1387,27 @@ class SparkContext(config: SparkConf) extends
Logging with
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r63991284
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -1724,6 +1745,22 @@ class SparkContext(config: SparkConf) extends
Logging with
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r63991184
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala
---
@@ -46,3 +46,33 @@ case class AddFile(path: String) extends
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/13212#discussion_r63991179
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala
---
@@ -46,3 +46,33 @@ case class AddFile(path: String) extends
Github user xwu0226 commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-220516455
cc @yhuai @hvanhovell @gatorsmile 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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-220516397
Can one of the admins verify this patch?
---
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 xwu0226 opened a pull request:
https://github.com/apache/spark/pull/13212
[SPARK-15431][SQL] Support LIST FILE(s)|JAR(s) command natively
## What changes were proposed in this pull request?
Currently command "ADD FILE|JAR " is supported natively
in
100 matches
Mail list logo