Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14910
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
enabled and wishes so, or if the
Github user ericl commented on the issue:
https://github.com/apache/spark/pull/14905
Ah, too bad then. 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
enabled and wishes
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14883#discussion_r77117696
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalog.scala
---
@@ -184,4 +184,9 @@ abstract class ExternalCatalog
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14911
**[Test build #64761 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64761/consoleFull)**
for PR 14911 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14883#discussion_r77117555
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalog.scala
---
@@ -184,4 +184,9 @@ abstract class ExternalCatalog
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14883
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 feature
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14864#discussion_r77117501
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/DataSourceScanExec.scala
---
@@ -156,24 +156,56 @@ case class FileSourceScanExec(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14883
**[Test build #64755 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64755/consoleFull)**
for PR 14883 at commit
Github user JoshRosen commented on the issue:
https://github.com/apache/spark/pull/14907
Please merge #14911 ahead of this so that I can bring this up-to-date with
that change. Merging in this order reduces the amount of work to backport
#14911.
---
If your project is set up for
GitHub user JoshRosen opened a pull request:
https://github.com/apache/spark/pull/14911
[SPARK-17355] Workaround for HIVE-14684 / HiveResultSetMetaData.isSigned
exception
## What changes were proposed in this pull request?
Attempting to use Spark SQL's JDBC data source
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14910
**[Test build #64760 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64760/consoleFull)**
for PR 14910 at commit
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/14841#discussion_r77117090
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/SortOrder.scala
---
@@ -61,6 +61,9 @@ case class SortOrder(child:
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/14910
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
enabled and wishes so, or
GitHub user tejasapatil opened a pull request:
https://github.com/apache/spark/pull/14910
[SPARK-17271] [SQL] Remove redundant `semanticEquals()` from `SortOrder`
## What changes were proposed in this pull request?
Removing `semanticEquals()` from `SortOrder` because it can
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14531#discussion_r77116198
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveDDLSuite.scala
---
@@ -660,6 +662,236 @@ class HiveDDLSuite
}
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14531#discussion_r77116211
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveDDLSuite.scala
---
@@ -660,6 +662,236 @@ class HiveDDLSuite
}
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14823
**[Test build #64759 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64759/consoleFull)**
for PR 14823 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14823
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64759/
Test FAILed.
---
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14883#discussion_r77115989
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalog.scala
---
@@ -184,4 +184,9 @@ abstract class ExternalCatalog
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14531
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64754/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14823
**[Test build #64759 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64759/consoleFull)**
for PR 14823 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14531
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 feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14531
**[Test build #64754 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64754/consoleFull)**
for PR 14531 at commit
Github user kevinyu98 commented on the issue:
https://github.com/apache/spark/pull/12646
@chenghao-intel I have updated the codes based on your comments. Thanks a
lot.
Sure, I will work on that jira, so the fix is to just remove the space,
nothing else, right? Will that
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/14909
okay, 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 have this feature
enabled and wishes so, or if
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14909
---
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 cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14876
closing, @maropu will take over
---
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
Github user cloud-fan closed the pull request at:
https://github.com/apache/spark/pull/14876
---
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 cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14909
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 feature
enabled and wishes
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14841#discussion_r77114998
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/SortOrder.scala
---
@@ -61,6 +61,9 @@ case class SortOrder(child:
Github user Sherry302 commented on the issue:
https://github.com/apache/spark/pull/14659
@steveloughran Thank you very much. I have updated the PR based on your
comments. Also, I have added an unit test.
---
If your project is set up for it, you can reply to this email and have
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14783
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 feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14783
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64756/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14783
**[Test build #64756 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64756/consoleFull)**
for PR 14783 at commit
Github user Sherry302 commented on the issue:
https://github.com/apache/spark/pull/14659
@srowen Thanks all the same.
---
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
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14868#discussion_r77114814
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/ApproximatePercentile.scala
---
@@ -0,0 +1,321 @@
+/*
+
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14452
**[Test build #64758 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64758/consoleFull)**
for PR 14452 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14659
**[Test build #64757 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64757/consoleFull)**
for PR 14659 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/14883#discussion_r77114469
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalog.scala
---
@@ -184,4 +184,9 @@ abstract class ExternalCatalog {
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/14883#discussion_r77114400
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalog.scala
---
@@ -184,4 +184,9 @@ abstract class ExternalCatalog {
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14868#discussion_r77114139
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/ApproximatePercentile.scala
---
@@ -0,0 +1,321 @@
+/*
+
Github user angolon commented on the issue:
https://github.com/apache/spark/pull/14710
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 this feature
enabled and wishes so,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14909
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 feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14909
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64752/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14909
**[Test build #64752 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64752/consoleFull)**
for PR 14909 at commit
Github user yinxusen commented on the issue:
https://github.com/apache/spark/pull/9
Thanks @sethah and @dbtsai, I'll fix them soon.
---
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 tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/14841#discussion_r77113690
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/SortOrder.scala
---
@@ -61,6 +61,9 @@ case class SortOrder(child:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14883#discussion_r77113584
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalog.scala
---
@@ -184,4 +184,9 @@ abstract class ExternalCatalog
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14883#discussion_r77113302
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalog.scala
---
@@ -184,4 +184,9 @@ abstract class ExternalCatalog
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14712#discussion_r77113222
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeTableCommand.scala
---
@@ -88,24 +85,53 @@ case class
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14712#discussion_r77113174
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/LogicalRelation.scala
---
@@ -52,7 +52,8 @@ case class LogicalRelation(
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14856
---
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 cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14712#discussion_r77113054
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeTableCommand.scala
---
@@ -88,24 +85,53 @@ case class
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14856
Thanks @keypointt for the PR and @junyangq @felixcheung for reviewing.
Merging this into master
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/14905
> sc.jobProgressListener.waitUntilExecutorsUp(2, 3)
It's not a public API. So I cannot use it in the repl
---
If your project is set up for it, you can reply to this email and have
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14823
LGTM except one minor comment.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14783
**[Test build #64756 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64756/consoleFull)**
for PR 14783 at commit
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14783
Jenkins, 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 this feature
enabled and
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14903
---
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 shivaram commented on the issue:
https://github.com/apache/spark/pull/14903
Merging this into 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 does not have this feature
Github user angolon commented on the issue:
https://github.com/apache/spark/pull/14710
...*sigh*
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14710
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 does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14710
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64751/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14710
**[Test build #64751 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64751/consoleFull)**
for PR 14710 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/14388
@mallman Thanks. I will not share that file.
---
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 issue:
https://github.com/apache/spark/pull/14712
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64750/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14712
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 feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14712
**[Test build #64750 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64750/consoleFull)**
for PR 14712 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14823#discussion_r77111446
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala
---
@@ -123,10 +108,7 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14908
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64747/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14908
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 feature
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14531#discussion_r7752
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveDDLSuite.scala
---
@@ -660,6 +662,236 @@ class HiveDDLSuite
}
}
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14908
**[Test build #64747 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64747/consoleFull)**
for PR 14908 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14900
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64749/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14900
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 feature
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14531#discussion_r77111051
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveDDLSuite.scala
---
@@ -660,6 +662,236 @@ class HiveDDLSuite
}
}
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14900
**[Test build #64749 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64749/consoleFull)**
for PR 14900 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14900
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 feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14900
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64748/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14900
**[Test build #64748 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64748/consoleFull)**
for PR 14900 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14859
@shivaram @felixcheung Thanks for your feedback.
I will also test if your comments are actually feasible or not (building
nightly & filtering commits).
Then, I will try to clean up
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14883#discussion_r77110718
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalog.scala
---
@@ -184,4 +184,9 @@ abstract class ExternalCatalog
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14858
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64753/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14858
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 feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14858
**[Test build #64753 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64753/consoleFull)**
for PR 14858 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14883
**[Test build #64755 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64755/consoleFull)**
for PR 14883 at commit
Github user ScrapCodes commented on the issue:
https://github.com/apache/spark/pull/14553
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 this feature
enabled and wishes
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14883#discussion_r77110077
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala
---
@@ -37,13 +38,13 @@ case class AddJarCommand(path: String)
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/10225#discussion_r77109895
--- Diff:
core/src/main/scala/org/apache/spark/storage/DiskBlockManager.scala ---
@@ -50,35 +50,98 @@ private[spark] class DiskBlockManager(conf: SparkConf,
Github user jpiper commented on a diff in the pull request:
https://github.com/apache/spark/pull/14861#discussion_r77109772
--- Diff: python/test_support/test_folder/test_folder2/hello.txt ---
@@ -0,0 +1 @@
+Hello World!
--- End diff --
I wanted to ensure that the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14531
**[Test build #64754 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64754/consoleFull)**
for PR 14531 at commit
Github user zjffdu commented on a diff in the pull request:
https://github.com/apache/spark/pull/14861#discussion_r77109177
--- Diff: python/test_support/test_folder/test_folder2/hello.txt ---
@@ -0,0 +1 @@
+Hello World!
--- End diff --
Sorry didn't notice this is
Github user zjffdu commented on a diff in the pull request:
https://github.com/apache/spark/pull/14861#discussion_r77109142
--- Diff: python/test_support/test_folder/test_folder2/hello.txt ---
@@ -0,0 +1 @@
+Hello World!
--- End diff --
Please remove this file
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/14712
Looks much better now.
---
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
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/14712#discussion_r77108572
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/LogicalRelation.scala
---
@@ -72,9 +73,11 @@ case class LogicalRelation(
Github user zenglinxi0615 commented on the issue:
https://github.com/apache/spark/pull/14686
sorry for long time no response.
yes, you are right, when you can change the sql from using '/temp/test.py'
to using 'python /temp/test.py', there's no need for changing the spark source
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/14452#discussion_r77108228
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/subquery/CommonSubquery.scala
---
@@ -0,0 +1,60 @@
+/*
+ * Licensed to the Apache
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14866
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64746/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14866
**[Test build #3242 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3242/consoleFull)**
for PR 14866 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14866
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 does not have this feature
1 - 100 of 631 matches
Mail list logo
