Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/14834#discussion_r78464438
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala
---
@@ -311,8 +350,28 @@ class LogisticRegression @Since("1.2.0")
Github user Stibbons commented on a diff in the pull request:
https://github.com/apache/spark/pull/15026#discussion_r78464418
--- Diff: python/pyspark/broadcast.py ---
@@ -75,7 +75,13 @@ def __init__(self, sc=None, value=None,
pickle_registry=None, path=None):
self
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15048
**[Test build #65283 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65283/consoleFull)**
for PR 15048 at commit
[`4c3c955`](https://github.com/apache/spark/commit/4
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15065
**[Test build #65282 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65282/consoleFull)**
for PR 15065 at commit
[`4c6bb0b`](https://github.com/apache/spark/commit/4
GitHub user zsxwing opened a pull request:
https://github.com/apache/spark/pull/15065
[SPARK-17463][Core]Add necessary memory barrier for accumulators
## What changes were proposed in this pull request?
Added `volatile` for fields that will be read in the heartbeat thread.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14467
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14467
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65274/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14467
**[Test build #65274 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65274/consoleFull)**
for PR 14467 at commit
[`6169c3c`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11105
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65276/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11105
**[Test build #65276 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65276/consoleFull)**
for PR 11105 at commit
[`491499d`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11105
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
e
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15048#discussion_r78461431
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/rules.scala
---
@@ -68,7 +68,7 @@ class ResolveDataSource(sparkSession: Spar
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15063
**[Test build #65281 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65281/consoleFull)**
for PR 15063 at commit
[`0da2f9b`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12601
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65275/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12601
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12601
**[Test build #65275 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65275/consoleFull)**
for PR 12601 at commit
[`7ef7a48`](https://github.com/apache/spark/commit/
Github user avulanov commented on the issue:
https://github.com/apache/spark/pull/13617
@JeremyNixon Sounds good!
---
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 wish
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15064
**[Test build #65280 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65280/consoleFull)**
for PR 15064 at commit
[`19a2d96`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15064
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15064
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65280/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15030
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15030
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65273/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15064
**[Test build #65280 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65280/consoleFull)**
for PR 15064 at commit
[`19a2d96`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13758
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15030
**[Test build #65273 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65273/consoleFull)**
for PR 15030 at commit
[`1e319d8`](https://github.com/apache/spark/commit/
GitHub user sitalkedia opened a pull request:
https://github.com/apache/spark/pull/15064
[SPARK-17509]]When wrapping catalyst datatype to Hive data type avoidâ¦
## What changes were proposed in this pull request?
When wrapping catalyst datatypes to Hive data type, wrap func
Github user sitalkedia commented on the issue:
https://github.com/apache/spark/pull/15064
cc - @rxin
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13758
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65277/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13758
**[Test build #65277 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65277/consoleFull)**
for PR 13758 at commit
[`45ae9bf`](https://github.com/apache/spark/commit/
Github user yhuai commented on the issue:
https://github.com/apache/spark/pull/15061
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 feature
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/14959
Also someone else more familiar with pyspark (I know Holden has already
looked), maybe @davies, should take a look.
---
If your project is set up for it, you can reply to this email and have your
re
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/14959
The conf code looks kinda nasty with the checks for whether a JVM has been
set or not... I guess part of it is mandatory because otherwise this wouldn't
work, but in particular, I'm not so sure the `
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15063
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15063
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65272/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15063
**[Test build #65272 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65272/consoleFull)**
for PR 15063 at commit
[`5a7183b`](https://github.com/apache/spark/commit/
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/14959#discussion_r78453870
--- Diff: python/pyspark/java_gateway.py ---
@@ -51,13 +51,16 @@ def launch_gateway():
on_windows = platform.system() == "Windows"
sc
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13617
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65278/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13617
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13617
**[Test build #65278 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65278/consoleFull)**
for PR 13617 at commit
[`509cb23`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15063
**[Test build #3255 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3255/consoleFull)**
for PR 15063 at commit
[`21b6b4d`](https://github.com/apache/spark/commit
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/13642#discussion_r78450919
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/metric/SQLMetricsSuite.scala
---
@@ -17,48 +17,18 @@
package org.apache.spark.sq
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15063
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15063
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65270/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15063
**[Test build #65270 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65270/consoleFull)**
for PR 15063 at commit
[`4b8c277`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15059
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65265/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15059
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15059
**[Test build #65265 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65265/consoleFull)**
for PR 15059 at commit
[`78f3733`](https://github.com/apache/spark/commit/
Github user JeremyNixon commented on the issue:
https://github.com/apache/spark/pull/13617
@avulanov I am interested - how about I replicate this PR at
github.com/avulanov/scalable-deeplearning and we discuss details there?
---
If your project is set up for it, you can reply to this
Github user ericl commented on the issue:
https://github.com/apache/spark/pull/15037
This 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 fe
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15062
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65269/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15062
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15062
**[Test build #65269 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65269/consoleFull)**
for PR 15062 at commit
[`931f156`](https://github.com/apache/spark/commit/
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15036
---
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 ena
Github user JoshRosen commented on the issue:
https://github.com/apache/spark/pull/15036
Thanks for the reviews. I'm going to merge this into 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 no
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14842
**[Test build #65279 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65279/consoleFull)**
for PR 14842 at commit
[`5153ce5`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14961
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14961
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65264/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14961
**[Test build #65264 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65264/consoleFull)**
for PR 14961 at commit
[`502ebf4`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13617
**[Test build #65278 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65278/consoleFull)**
for PR 13617 at commit
[`509cb23`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13758
**[Test build #65277 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65277/consoleFull)**
for PR 13758 at commit
[`45ae9bf`](https://github.com/apache/spark/commit/4
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12601
**[Test build #65275 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65275/consoleFull)**
for PR 12601 at commit
[`7ef7a48`](https://github.com/apache/spark/commit/7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11105
**[Test build #65276 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65276/consoleFull)**
for PR 11105 at commit
[`491499d`](https://github.com/apache/spark/commit/4
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14467
**[Test build #65274 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65274/consoleFull)**
for PR 14467 at commit
[`6169c3c`](https://github.com/apache/spark/commit/6
Github user skonto commented on a diff in the pull request:
https://github.com/apache/spark/pull/14644#discussion_r78438011
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackend.scala
---
@@ -103,6 +103,7 @@ private[spark] class
Github user JustinPihony commented on the issue:
https://github.com/apache/spark/pull/12601
@srowen Documentation added.
---
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 a
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15024
[`showDataSourceTableOptions`](https://github.com/apache/spark/blob/c0ae6bc6ea38909730fad36e653d3c7ab0a84b44/sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala#L866-L873)
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15030
**[Test build #65273 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65273/consoleFull)**
for PR 15030 at commit
[`1e319d8`](https://github.com/apache/spark/commit/1
Github user marmbrus commented on the issue:
https://github.com/apache/spark/pull/15023
Thanks for spending the time to backport this, but it does seem a little
risky to include changes to the configuration system in a maintenance release.
As such, I'd probably error on the side of c
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/15030#discussion_r78435023
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/limit.scala ---
@@ -148,8 +148,8 @@ case class TakeOrderedAndProjectExec(
localT
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15060
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15060
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65266/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15060
**[Test build #65266 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65266/consoleFull)**
for PR 15060 at commit
[`e89aef5`](https://github.com/apache/spark/commit/
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15030#discussion_r78434025
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/limit.scala ---
@@ -148,8 +148,8 @@ case class TakeOrderedAndProjectExec(
loc
Github user coderfi commented on the issue:
https://github.com/apache/spark/pull/14857
Awesome, we ran into this problem as well, and finally had some bandwidth
to track down the cause well enough to be able to search for this pull request.
Looking forward to test this in our fork.
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/15063#discussion_r78433588
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/debug/package.scala ---
@@ -107,18 +109,20 @@ package object debug {
case class Debug
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15024#discussion_r78433410
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -447,17 +461,10 @@ private[spark] class HiveExternalCatalog(c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15063
**[Test build #65272 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65272/consoleFull)**
for PR 15063 at commit
[`5a7183b`](https://github.com/apache/spark/commit/5
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15063#discussion_r78432833
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/debug/package.scala ---
@@ -107,18 +109,20 @@ package object debug {
case class Deb
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/14467
Ping @MLnick / @srowen ?
---
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 davies commented on the issue:
https://github.com/apache/spark/pull/15026
Once we could log the original stacktrace, this looks good to me.
---
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 davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/15026#discussion_r78432383
--- Diff: python/pyspark/cloudpickle.py ---
@@ -109,6 +109,15 @@ def dump(self, obj):
if 'recursion' in e.args[0]:
msg =
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/15026#discussion_r78432125
--- Diff: python/pyspark/broadcast.py ---
@@ -75,7 +75,13 @@ def __init__(self, sc=None, value=None,
pickle_registry=None, path=None):
self._
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/15036#discussion_r78430377
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala
---
@@ -1316,21 +1303,31 @@ private[spark] class BlockManager(
// The bl
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15056
---
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 ena
Github user JoshRosen commented on the issue:
https://github.com/apache/spark/pull/15056
LGTM as well, so I'm going to merge this to master and branch-2.0
(2.0.1-SNAPSHOT). Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/15056#discussion_r78426904
--- Diff:
core/src/main/scala/org/apache/spark/storage/memory/MemoryStore.scala ---
@@ -663,31 +663,43 @@ private[spark] class MemoryStore(
private[storag
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10655
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10655
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65267/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10655
**[Test build #65267 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65267/consoleFull)**
for PR 10655 at commit
[`5a0daf6`](https://github.com/apache/spark/commit/
Github user JoshRosen commented on the issue:
https://github.com/apache/spark/pull/15063
I think that we may also want to do this for `BlockStatusesAccumulator`
---
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 proj
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15056#discussion_r78425128
--- Diff:
core/src/main/scala/org/apache/spark/storage/memory/MemoryStore.scala ---
@@ -663,31 +663,43 @@ private[spark] class MemoryStore(
private[st
Github user xwu0226 commented on the issue:
https://github.com/apache/spark/pull/14842
@ericl Thanks so much for the detailed review and suggestions. I will fix
the last comments.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user ericl commented on the issue:
https://github.com/apache/spark/pull/14842
A few more minor comments but otherwise the prefix parts look good to me!
---
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 projec
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10655
**[Test build #65267 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65267/consoleFull)**
for PR 10655 at commit
[`5a0daf6`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15062
**[Test build #65269 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65269/consoleFull)**
for PR 15062 at commit
[`931f156`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15061
**[Test build #65268 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65268/consoleFull)**
for PR 15061 at commit
[`1224e75`](https://github.com/apache/spark/commit/1
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/14842#discussion_r78424378
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/SortOrder.scala
---
@@ -21,26 +21,44 @@ import org.apache.spark.sql.catalyst.I
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15063
**[Test build #3255 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3255/consoleFull)**
for PR 15063 at commit
[`21b6b4d`](https://github.com/apache/spark/commit/
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/14842#discussion_r78424255
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/SortOrder.scala
---
@@ -71,12 +95,38 @@ case class SortPrefix(child: SortOrder
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/14842#discussion_r78424193
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/SortOrder.scala
---
@@ -71,12 +95,38 @@ case class SortPrefix(child: SortOrder
201 - 300 of 559 matches
Mail list logo
