GitHub user zjffdu opened a pull request:
https://github.com/apache/spark/pull/17367
[MINOR][PYSPARK] Remove _inferSchema in context.py
## What changes were proposed in this pull request?
_inferSchema is not used in context.py, all the things have been moved to
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/17347
okay!
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17014
**[Test build #74932 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74932/testReport)**
for PR 17014 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17347
can you send a new PR for branch 2.1? 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
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17293
Uh.. I am actually not sure if we want the non-nullability per
https://github.com/apache/spark/pull/14124#issuecomment-259615392.
I am willing to help test and verify if it is okay for
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17191#discussion_r107066194
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -836,17 +836,30 @@ class Analyzer(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17136
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74924/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17136
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17136
**[Test build #74924 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74924/testReport)**
for PR 17136 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17347
thanks, 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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17347
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17276
**[Test build #74931 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74931/testReport)**
for PR 17276 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17312
**[Test build #74930 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74930/testReport)**
for PR 17312 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17276
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74917/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17276
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17276
**[Test build #74917 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74917/testReport)**
for PR 17276 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17014
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74925/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17014
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 jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/17342#discussion_r107064584
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SharedState.scala ---
@@ -148,6 +149,8 @@ private[sql] class SharedState(val
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17014
**[Test build #74925 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74925/testReport)**
for PR 17014 at commit
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/17342#discussion_r107064456
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -2767,3 +2767,24 @@ private[spark] class CircularBuffer(sizeInBytes: Int
= 10240)
Github user weiqingy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17342#discussion_r107063651
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -2767,3 +2767,24 @@ private[spark] class CircularBuffer(sizeInBytes: Int
= 10240)
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16626#discussion_r107063564
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalogSuite.scala
---
@@ -455,8 +455,10 @@ abstract class
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16626#discussion_r107063558
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalogSuite.scala
---
@@ -455,8 +455,10 @@ abstract class
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16626#discussion_r107063379
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -343,15 +343,11 @@ class SessionCatalog(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17014
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74921/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17014
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/16626
**[Test build #74929 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74929/testReport)**
for PR 16626 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17014
**[Test build #74921 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74921/testReport)**
for PR 17014 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16626#discussion_r107063198
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -296,6 +311,47 @@ class SessionCatalog(
Github user xwu0226 commented on the issue:
https://github.com/apache/spark/pull/16626
@cloud-fan @gatorsmile Thanks again! I updated the code based @cloud-fan 's
review.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17302#discussion_r107062991
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ExistingRDD.scala ---
@@ -70,7 +70,20 @@ object RDDConversions {
object ExternalRDD
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17312
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/17312
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74916/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17312
**[Test build #74916 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74916/testReport)**
for PR 17312 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17339
---
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/17339
thanks, 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
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17353
Yeah. The counts can help us understand the pruning rate of the search
space. When CBO join reordering is very slow, we can check the counts.
---
If your project is set up for it, you can reply
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14038
**[Test build #74927 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74927/testReport)**
for PR 14038 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14038
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74927/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14038
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17354
**[Test build #74928 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74928/testReport)**
for PR 17354 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17354
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17354
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/17354
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74915/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17354
**[Test build #74915 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74915/testReport)**
for PR 17354 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17312
**[Test build #74926 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74926/testReport)**
for PR 17312 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14038
**[Test build #74927 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74927/testReport)**
for PR 14038 at commit
Github user ehsun7b commented on a diff in the pull request:
https://github.com/apache/spark/pull/17314#discussion_r107061443
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/r/AFTSurvivalRegressionWrapper.scala
---
@@ -30,6 +32,7 @@ import
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/17334
"Change the exception log to add RDD id of the related the block".
---
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 jinxing64 commented on the issue:
https://github.com/apache/spark/pull/17312
Sure, that would be cool :) Thanks again you can help review this.
---
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 leifker commented on the issue:
https://github.com/apache/spark/pull/17306
Interesting, let me think about this a bit. I think that there is probably
a better api around this approach for sure.
---
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/17352
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74923/
Test PASSed.
---
Github user jianran commented on the issue:
https://github.com/apache/spark/pull/17313
do you agree remove the comment is more appreciable? can i only remove the
comment?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17352
**[Test build #74923 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74923/testReport)**
for PR 17352 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17352
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/17352
**[Test build #74923 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74923/testReport)**
for PR 17352 at commit
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/16605
@cloud-fan ping
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17014
**[Test build #74925 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74925/testReport)**
for PR 17014 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17136
**[Test build #74924 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74924/testReport)**
for PR 17136 at commit
Github user jianran commented on the issue:
https://github.com/apache/spark/pull/17334
@jerryshao what's title do you think is better?
---
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 cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17353
debug log SGTM
---
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 jerryshao commented on the issue:
https://github.com/apache/spark/pull/17312
IMO "(2 executors)" should be enough :).
---
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/17191
**[Test build #74922 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74922/testReport)**
for PR 17191 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17014
**[Test build #74921 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74921/testReport)**
for PR 17014 at commit
Github user CodingCat commented on the issue:
https://github.com/apache/spark/pull/14617
I agree with the checkpoint based solution , thanks for asking
---
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
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16626#discussion_r107057509
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -2178,4 +2178,138 @@ abstract class DDLSuite extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16209
**[Test build #74920 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74920/testReport)**
for PR 16209 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16626#discussion_r107057320
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -296,6 +311,51 @@ class SessionCatalog(
}
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/17347
@cloud-fan Could you check this? 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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17361
**[Test build #74919 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74919/testReport)**
for PR 17361 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17354
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17315
**[Test build #74918 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74918/testReport)**
for PR 17315 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17315#discussion_r107055987
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/FailureSafeParser.scala
---
@@ -0,0 +1,80 @@
+/*
+ * Licensed to the
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16971
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17362
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74912/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17362
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/17362
**[Test build #74912 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74912/testReport)**
for PR 17362 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16971
Since it is close to code freeze, I am first merging this PR. If any more
comments, we can resolve them in the follow-up PR.
---
If your project is set up for it, you can reply to this email
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17315#discussion_r107055187
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/FailureSafeParser.scala
---
@@ -0,0 +1,80 @@
+/*
+ * Licensed to the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17219
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74914/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17219
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17219
**[Test build #74914 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74914/testReport)**
for PR 17219 at commit
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16626#discussion_r107053921
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -2178,4 +2178,138 @@ abstract class DDLSuite extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17276
**[Test build #74917 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74917/testReport)**
for PR 17276 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17315#discussion_r107053686
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/FailureSafeParser.scala
---
@@ -0,0 +1,80 @@
+/*
+ * Licensed to the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17295
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/17295
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74911/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17295
**[Test build #74911 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74911/testReport)**
for PR 17295 at commit
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/17267
I'll take a look at reviewing this later on this week @uncleGen. Two minor
thing that we can do in the meantime is make the JIRA description a bit clearer
as to what the proposed change is, the
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/17352
This looks reasonable, however there is a conflict, can you update this to
the latest master?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/17312
@rxin @jerryshao
I uploaded another screenshot and give a short description there.
Now it is (2 executors supplied).
---
If your project is set up for it, you can reply to this email and
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/17312
---
If your project is set up for it, you can reply to this email
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16626#discussion_r107051682
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveDDLSuite.scala
---
@@ -1860,4 +1861,115 @@ class HiveDDLSuite
}
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17312
**[Test build #74916 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74916/testReport)**
for PR 17312 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17361
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/17361
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74910/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17361
**[Test build #74910 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74910/testReport)**
for PR 17361 at commit
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16626#discussion_r107051308
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -2178,4 +2178,138 @@ abstract class DDLSuite extends
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16626#discussion_r107050210
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveDDLSuite.scala
---
@@ -1860,4 +1861,115 @@ class HiveDDLSuite
}
101 - 200 of 635 matches
Mail list logo
