Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20525#discussion_r167159659
--- Diff: docs/sql-programming-guide.md ---
@@ -1930,6 +1930,9 @@ working with timestamps in `pandas_udf`s to get the
best performance, see
- Lit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20516#discussion_r167159549
--- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
@@ -59,6 +59,7 @@ abstract class SparkFunSuite
protected val enableAutoThread
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/20525#discussion_r167159534
--- Diff: docs/sql-programming-guide.md ---
@@ -1930,6 +1930,9 @@ working with timestamps in `pandas_udf`s to get the
best performance, see
- L
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20449#discussion_r167158719
--- Diff:
core/src/main/scala/org/apache/spark/shuffle/BlockStoreShuffleReader.scala ---
@@ -104,9 +104,16 @@ private[spark] class BlockStoreShuffleReader
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/20525
@tdas @brkyvz Do we still need the fix for 0-partition DataFrame in
Structured Streaming after this change?
---
-
To unsubscribe
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20525#discussion_r167158557
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/test/DataFrameReaderWriterSuite.scala
---
@@ -301,7 +301,6 @@ class DataFrameReaderWriterSuite ext
Github user heary-cao commented on a diff in the pull request:
https://github.com/apache/spark/pull/20516#discussion_r167158512
--- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
@@ -59,6 +59,7 @@ abstract class SparkFunSuite
protected val enableAutoThread
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20525#discussion_r167158389
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/FileFormatWriterSuite.scala
---
@@ -19,6 +19,7 @@ package org.apache.spark.s
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20525#discussion_r167158260
--- Diff: docs/sql-programming-guide.md ---
@@ -1930,6 +1930,9 @@ working with timestamps in `pandas_udf`s to get the
best performance, see
- Lit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20516#discussion_r167156814
--- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
@@ -59,6 +59,7 @@ abstract class SparkFunSuite
protected val enableAutoThread
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20555
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20555
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87243/
Test PASSed.
---
Github user heary-cao commented on the issue:
https://github.com/apache/spark/pull/20516
I try with mavn(using command line) to test case , it is right. thanks.
Then, whether we add System.setProperty("spark.testing", "true") in
SparkFunSuite to slove the IDE test tool problem. Be
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20555
**[Test build #87243 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87243/testReport)**
for PR 20555 at commit
[`b26ffce`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20477
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87247/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20477
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20477
**[Test build #87247 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87247/testReport)**
for PR 20477 at commit
[`0cc0600`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20490
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20490
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87244/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20490
**[Test build #87244 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87244/testReport)**
for PR 20490 at commit
[`e9964ca`](https://github.com/apache/spark/commit/e
Github user rekhajoshm closed the pull request at:
https://github.com/apache/spark/pull/20501
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user rekhajoshm commented on the issue:
https://github.com/apache/spark/pull/20501
Ack. thanks for the update @felixcheung @srowen Closing this.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.o
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20499
Yup, I should fix the guide for 2.2 anyway :-) Will open a backport tonight
KST.
---
-
To unsubscribe, e-mail: reviews-unsub
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20525
**[Test build #87250 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87250/testReport)**
for PR 20525 at commit
[`30e5aa5`](https://github.com/apache/spark/commit/30
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20525
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20525
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/743/
Test
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20537
LGTM too
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spa
Github user dilipbiswal closed the pull request at:
https://github.com/apache/spark/pull/20551
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.or
Github user advancedxy commented on the issue:
https://github.com/apache/spark/pull/20449
@jerryshao @cloud-fan I have updated my code. Do you have any other
concerns?
---
-
To unsubscribe, e-mail: reviews-unsubscr.
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/20525
@cloud-fan @gatorsmile Done.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-m
Github user rekhajoshm closed the pull request at:
https://github.com/apache/spark/pull/20378
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user rekhajoshm commented on the issue:
https://github.com/apache/spark/pull/20378
@HyukjinKwon @holdenk @ueshin @viirya @icexelloss @felixcheung @BryanCutler
and @MrBago - This was one of the possible approach that I was running by you.
I have proposed another approach at #205
Github user rekhajoshm commented on the issue:
https://github.com/apache/spark/pull/20378
@HyukjinKwon Identifying docstyle failures does not help much as it is not
straightforward to exclude in this version.
---
-
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20525
no we can't merge 2 PRs together. Please pick one of your PRs and put all
the changes there, thanks!
---
-
To unsubscribe, e-m
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20556
**[Test build #87249 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87249/testReport)**
for PR 20556 at commit
[`ee14cf7`](https://github.com/apache/spark/commit/ee
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20556
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20556
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/742/
Test
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/20525
@cloud-fan Actually i had already created the doc pr in the morning using
the same JIRA number. Whenchen, if we want to have both the changes in the same
commit , will we be able to do it when w
Github user rekhajoshm commented on a diff in the pull request:
https://github.com/apache/spark/pull/20378#discussion_r167148657
--- Diff: dev/lint-python ---
@@ -83,6 +84,53 @@ else
rm "$PEP8_REPORT_PATH"
fi
+ Python Document Style Checks
+
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20499
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20499
thanks, merging to master/2.3!
Can you send a new PR for 2.2? it conflicts...
---
-
To unsubscribe, e-mail: reviews-un
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20387#discussion_r167147910
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2Relation.scala
---
@@ -17,17 +17,130 @@
package or
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20557
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20557
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user guoxiaolongzte opened a pull request:
https://github.com/apache/spark/pull/20557
[SPARK-23364][SQL]'desc table' command in spark-sql add column head display
## What changes were proposed in this pull request?
Use 'desc partition_table' command in spark-sql clien
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20359
Thank you for merging, @cloud-fan .
And thank you again, @HyukjinKwon , @gatorsmile , and @srowen !
---
-
To unsubscrib
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20556
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20556
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87248/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20556
**[Test build #87248 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87248/testReport)**
for PR 20556 at commit
[`85ca69d`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20556
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/741/
Test
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20556
**[Test build #87248 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87248/testReport)**
for PR 20556 at commit
[`85ca69d`](https://github.com/apache/spark/commit/85
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20556
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user squito commented on the issue:
https://github.com/apache/spark/pull/20532
I can see why you want this sometimes, but I'm trying to figure out if its
really valuable for users in general. You could always add a custom listener
to log this info. It would go into separate f
GitHub user rekhajoshm opened a pull request:
https://github.com/apache/spark/pull/20556
[SPARK-23367][Build] Include python document style checking
## What changes were proposed in this pull request?
Include python document style checking.
This PR includes the pydocstyle che
Github user ivoson commented on a diff in the pull request:
https://github.com/apache/spark/pull/20244#discussion_r167145734
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/DAGSchedulerSuite.scala ---
@@ -2399,6 +2424,121 @@ class DAGSchedulerSuite extends SparkFunSuite wi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20554
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20554
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87239/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20554
**[Test build #87239 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87239/testReport)**
for PR 20554 at commit
[`05c9d20`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20554
Build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20554
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87238/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20554
**[Test build #87238 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87238/testReport)**
for PR 20554 at commit
[`3ed2a50`](https://github.com/apache/spark/commit/3
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20477
**[Test build #87247 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87247/testReport)**
for PR 20477 at commit
[`0cc0600`](https://github.com/apache/spark/commit/0c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20477
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/740/
Test
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20477
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20477
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20516
Can you try with SBT(using command line)? Usually we don't trust the test
result of IDE.
---
-
To unsubscribe, e-mail: reviews
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20525
I think it's better to have the doc change in the same PR, then it's more
clear which patch caused the behavior change.
---
-
Github user heary-cao commented on the issue:
https://github.com/apache/spark/pull/20516
sure,
Operation environment: IDEA test tool.
test case: test("can bind to a specific port")
Test code:
val maxRetries = portMaxRetries(conf)
println("maxRetries:" + maxRetri
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20387
> We've added a resolution rule from UnresolvedRelation to
DataSourceV2Relation that uses our implementation. UnresolvedRelation needs to
pass its TableIdentifier to the v2 relation, which is why
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20545
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/739/
Test
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20545
**[Test build #87246 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87246/testReport)**
for PR 20545 at commit
[`664a62c`](https://github.com/apache/spark/commit/66
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20545
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20545
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20387#discussion_r167142433
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2Relation.scala
---
@@ -17,17 +17,130 @@
package or
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20477
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87242/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20477
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20477
**[Test build #87242 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87242/testReport)**
for PR 20477 at commit
[`0cc0600`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20552
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87241/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20552
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20552
**[Test build #87241 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87241/testReport)**
for PR 20552 at commit
[`a33a35c`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20303
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20303
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87236/
Test PASSed.
---
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20387#discussion_r167141001
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2Relation.scala
---
@@ -17,17 +17,130 @@
package or
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20303
**[Test build #87236 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87236/testReport)**
for PR 20303 at commit
[`603c6d5`](https://github.com/apache/spark/commit/6
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20521
**[Test build #87245 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87245/testReport)**
for PR 20521 at commit
[`6bc913f`](https://github.com/apache/spark/commit/6b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20521
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/738/
Test
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20521
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20541
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20521#discussion_r167140801
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveExplainSuite.scala
---
@@ -128,32 +128,6 @@ class HiveExplainSuite extends Quer
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20541
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87237/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20541
**[Test build #87237 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87237/testReport)**
for PR 20541 at commit
[`4f5d46b`](https://github.com/apache/spark/commit/4
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20359
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20359
thanks, merging to master/2.3!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-ma
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20516
are you sure `./project/SparkBuild.scala:795: javaOptions in Test +=
"-Dspark.testing=1"` only affect non-test code path? Then we have a lot of
places to fix.
---
--
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20545
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20545
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87240/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20545
**[Test build #87240 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87240/testReport)**
for PR 20545 at commit
[`664a62c`](https://github.com/apache/spark/commit/6
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/20244#discussion_r167138603
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/DAGSchedulerSuite.scala ---
@@ -2399,6 +2424,121 @@ class DAGSchedulerSuite extends SparkFunSuite wi
Github user heary-cao commented on a diff in the pull request:
https://github.com/apache/spark/pull/20516#discussion_r167137766
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -,7 +,7 @@ private[spark] object Utils extends Logging {
*/
d
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20490#discussion_r167137165
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/writer/DataSourceWriter.java
---
@@ -62,6 +62,16 @@
*/
DataWriterFactory creat
1 - 100 of 573 matches
Mail list logo
