Github user bersprockets commented on the issue:
https://github.com/apache/spark/pull/20638
Do we want to merge this or does it need more eyes?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For a
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20638
Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20678#discussion_r17186
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -1986,55 +1986,89 @@ def toPandas(self):
timezone = None
if self.sql_ctx
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20678#discussion_r17018
--- Diff: docs/sql-programming-guide.md ---
@@ -1689,6 +1689,10 @@ using the call `toPandas()` and when creating a
Spark DataFrame from a Pandas Da
`cr
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20638
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user bersprockets commented on the issue:
https://github.com/apache/spark/pull/20638
@HyukjinKwon Oh, thanks. I guess I learned what "approved these changes"
means.
---
-
To unsubscribe, e-mail: reviews-unsub
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20522
Sure. I'll try, @gatorsmile . It'll take some time for me.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.o
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/15670
So yeah if anyone has the spoons to take this over it _should_ just be
fixing the merge conflicts. Feel free to ping me on the PR.
---
-
Github user attilapiros commented on the issue:
https://github.com/apache/spark/pull/20046
cc @gatorsmile @hvanhovell
@jiangxb1987 your review comments are applied, is there something else I
should work on regarding this PR?
---
Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/20676
This is for compatibility reasons. The memory management also support
legacy memory management (`StaticMemoryManager`). In `StaticMemoryManager`, the
storage memory and unroll memory is managed sep
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/19876
So now that it looks like 2.3 is pretty much wrapped up do folks have any
thoughts? @mlnick @jkbradley @sethah ?
---
-
To unsubs
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/18339
Lets see what @BryanCutler thinks
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-m
Github user ConeyLiu commented on a diff in the pull request:
https://github.com/apache/spark/pull/20676#discussion_r171115071
--- Diff:
core/src/main/scala/org/apache/spark/storage/memory/MemoryStore.scala ---
@@ -246,18 +246,18 @@ private[spark] class MemoryStore(
va
Github user 10110346 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20676#discussion_r171115245
--- Diff:
core/src/main/scala/org/apache/spark/storage/memory/MemoryStore.scala ---
@@ -246,18 +246,18 @@ private[spark] class MemoryStore(
va
GitHub user henryr opened a pull request:
https://github.com/apache/spark/pull/20687
[SPARK-25000][SQL] Fix complex type simplification rules to apply to entire
plan
## What changes were proposed in this pull request?
Complex type simplification optimizer rules were not app
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20687
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/1121/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20687
Merged build finished. Test PASSed.
---
-
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/20687
**[Test build #87739 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87739/testReport)**
for PR 20687 at commit
[`f446fa2`](https://github.com/apache/spark/commit/f4
Github user 10110346 commented on the issue:
https://github.com/apache/spark/pull/20676
In `StaticMemoryManager`, the storage memory and unroll memory is managed
separately, but, unroll memory is also storage memory, so we do not need
release unroll memory really,Just need to release
Github user 10110346 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20676#discussion_r171116974
--- Diff:
core/src/main/scala/org/apache/spark/storage/memory/MemoryStore.scala ---
@@ -246,18 +246,18 @@ private[spark] class MemoryStore(
va
Github user caneGuy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20667#discussion_r171121778
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManagerId.scala
---
@@ -132,10 +133,15 @@ private[spark] object BlockManagerId {
getCac
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20666
**[Test build #87735 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87735/testReport)**
for PR 20666 at commit
[`daa326d`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20666
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87735/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20666
Merged build finished. Test PASSed.
---
-
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/20295
**[Test build #87736 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87736/testReport)**
for PR 20295 at commit
[`9ed3779`](https://github.com/apache/spark/commit/9
Github user zjffdu commented on the issue:
https://github.com/apache/spark/pull/13599
I am afraid I would not be present in Strata SJ, I live in Shanghai China,
and may not be able to travel at time.
---
-
To unsub
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20295
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/20295
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87736/
Test PASSed.
---
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/13599
Totally reasonable, how about if we did a group hangout or similar and then
summarized our discussions back to the PR? It could be after Strata and I can
get up early or go to bed late as needed for
Github user zjffdu commented on the issue:
https://github.com/apache/spark/pull/13599
That would be awesome.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user Ngone51 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20676#discussion_r171124468
--- Diff:
core/src/main/scala/org/apache/spark/storage/memory/MemoryStore.scala ---
@@ -246,18 +246,18 @@ private[spark] class MemoryStore(
val
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20666
Merged to master and branch-2.3.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/19108
@sethah Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20666
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/20676
Yeah, I see that. I'm not sure it's OK to change. But I think we should
follow the interface design, not the underlying implementation.
---
--
Github user Ngone51 commented on the issue:
https://github.com/apache/spark/pull/20676
Hi, @ConeyLiu. I don't think it's about compatibility. Because both
```StaticMemoryManager``` and ```UnifiedMemoryManager``` call the same function
for release unroll memory, which is ```releaseUnro
GitHub user jerryshao opened a pull request:
https://github.com/apache/spark/pull/20688
[SPARK-23096][SS] Migrate rate source to V2
## What changes were proposed in this pull request?
This PR migrate micro batch rate source to V2 API and rewrite UTs to suite
V2 test.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20688
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/20688
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/1122/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20688
**[Test build #87740 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87740/testReport)**
for PR 20688 at commit
[`538223e`](https://github.com/apache/spark/commit/53
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20666
Thanks @HyukjinKwon @cloud-fan!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user Ngone51 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20676#discussion_r171128021
--- Diff:
core/src/main/scala/org/apache/spark/storage/memory/MemoryStore.scala ---
@@ -246,18 +246,18 @@ private[spark] class MemoryStore(
val
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20685
**[Test build #87741 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87741/testReport)**
for PR 20685 at commit
[`110c851`](https://github.com/apache/spark/commit/11
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/20639
@zsxwing as well.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20685
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/1123/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20685
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/20639
this is ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18906
**[Test build #87742 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87742/testReport)**
for PR 18906 at commit
[`e94960c`](https://github.com/apache/spark/commit/e9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18906
**[Test build #87742 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87742/testReport)**
for PR 18906 at commit
[`e94960c`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18906
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87742/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18906
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/20681
**[Test build #87737 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87737/testReport)**
for PR 20681 at commit
[`21c3374`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20681
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/20681
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87737/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20679
**[Test build #87738 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87738/testReport)**
for PR 20679 at commit
[`b37f24f`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20679
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/20679
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87738/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18906
**[Test build #87743 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87743/testReport)**
for PR 18906 at commit
[`e6e6dbf`](https://github.com/apache/spark/commit/e6
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20687
**[Test build #87739 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87739/testReport)**
for PR 20687 at commit
[`f446fa2`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20687
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/20687
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87739/
Test FAILed.
---
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20676
I don't think unroll memory is storage memory from the interface. It's only
true for unified memory manager. I'm -1 on this change unless you can convince
the community to remove the static memory
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20667
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apa
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20667
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spa
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13599
Merged build finished. Test PASSed.
---
-
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/20667
**[Test build #87744 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87744/testReport)**
for PR 20667 at commit
[`3379899`](https://github.com/apache/spark/commit/33
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20667
add to whitelist
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13599
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/1124/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13599
**[Test build #87745 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87745/testReport)**
for PR 13599 at commit
[`3da68c7`](https://github.com/apache/spark/commit/3d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20667
**[Test build #87746 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87746/testReport)**
for PR 20667 at commit
[`3379899`](https://github.com/apache/spark/commit/33
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20667#discussion_r171136135
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManagerId.scala
---
@@ -132,10 +133,17 @@ private[spark] object BlockManagerId {
ge
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13599
**[Test build #87745 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87745/testReport)**
for PR 13599 at commit
[`3da68c7`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13599
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/13599
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87745/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20681
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/1125/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20681
Merged build finished. Test PASSed.
---
-
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/20681
**[Test build #87747 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87747/testReport)**
for PR 20681 at commit
[`999f86f`](https://github.com/apache/spark/commit/99
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20667
**[Test build #87748 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87748/testReport)**
for PR 20667 at commit
[`bf79f4d`](https://github.com/apache/spark/commit/bf
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20678#discussion_r171138898
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -1986,55 +1986,89 @@ def toPandas(self):
timezone = None
if self.sql
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20678#discussion_r171139748
--- Diff: docs/sql-programming-guide.md ---
@@ -1689,6 +1689,10 @@ using the call `toPandas()` and when creating a
Spark DataFrame from a Pandas Da
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20678
Will try to clean up soon.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20688
**[Test build #87740 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87740/testReport)**
for PR 20688 at commit
[`538223e`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20688
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/20688
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87740/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13599
Merged build finished. Test PASSed.
---
-
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/13599
**[Test build #87749 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87749/testReport)**
for PR 13599 at commit
[`86484d6`](https://github.com/apache/spark/commit/86
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13599
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/1126/
Tes
Github user Ngone51 commented on the issue:
https://github.com/apache/spark/pull/20667
Hi, @caneGuy , sorry for my previous comment as I mixed up ```BlockId```
with ```BlockManagerId```, and leave some wrong comments. And thanks for your
reply.
Back to now, I have the same qu
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20647
**[Test build #87750 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87750/testReport)**
for PR 20647 at commit
[`c5af52e`](https://github.com/apache/spark/commit/c5
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20647
Hi @rdblue , I've opened https://issues.apache.org/jira/browse/SPARK-23531
to include the type info. I'd like to do it later as it's a general problem in
Spark SQL and many plans need to be update
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20647
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/1127/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20647
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/13599
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/13599
**[Test build #87749 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87749/testReport)**
for PR 13599 at commit
[`86484d6`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13599
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87749/
Test FAILed.
---
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20647#discussion_r171143887
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2Relation.scala
---
@@ -107,19 +104,36 @@ case class DataSourc
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20667
In case the same `BlockManagerId` being created multiple times, this cache
will ensure we always use the first one that is created, which make it possible
for the rest `BlockManagerId` instances
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20647#discussion_r171143986
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2StringFormat.scala
---
@@ -0,0 +1,94 @@
+/*
+ * Licen
Github user Ngone51 commented on the issue:
https://github.com/apache/spark/pull/20667
Hi, @jiangxb1987 , thanks for your kindly explanation.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For add
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20685
**[Test build #87741 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87741/testReport)**
for PR 20685 at commit
[`110c851`](https://github.com/apache/spark/commit/1
301 - 400 of 462 matches
Mail list logo
