Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14568
**[Test build #63469 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63469/consoleFull)**
for PR 14568 at commit
[`f78d6aa`](https://github.com/apache/spark/commit/f
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/12990
I'll create a new pr based on this one with a higher default max tasks
(maybe 10,000?) and an on/off config when I have time this week
---
If your project is set up for it, you can reply to this
Github user petermaxlee commented on the issue:
https://github.com/apache/spark/pull/14472
I have updated this based on review feedback. The harness is now using
hiveResultString() rather than show(), similar to the existing Hive
compatibility test.
---
If your project is set up fo
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14568
**[Test build #63459 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63459/consoleFull)**
for PR 14568 at commit
[`46ea8a4`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14568
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/14568
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63459/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14568
**[Test build #63470 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63470/consoleFull)**
for PR 14568 at commit
[`1107182`](https://github.com/apache/spark/commit/1
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14472
**[Test build #63471 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63471/consoleFull)**
for PR 14472 at commit
[`0359756`](https://github.com/apache/spark/commit/0
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14527
Hi, @cloud-fan .
Could you review this PR when you have some time?
---
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 p
Github user ajbozarth commented on a diff in the pull request:
https://github.com/apache/spark/pull/12990#discussion_r74151754
--- Diff:
core/src/main/scala/org/apache/spark/ui/jobs/JobProgressListener.scala ---
@@ -137,6 +138,17 @@ class JobProgressListener(conf: SparkConf) extend
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14258#discussion_r74152794
--- Diff: R/pkg/R/install.R ---
@@ -0,0 +1,230 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor lice
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14568
**[Test build #63472 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63472/consoleFull)**
for PR 14568 at commit
[`8198d9c`](https://github.com/apache/spark/commit/8
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14397
Could you review this PR again, @hvanhovell ?
Last time, I did a huge mistake, but now I think I fixed correct according
to your advice.
---
If your project is set up for it, you can repl
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14568
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63472/
Test FAILed.
---
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/14397#discussion_r74153582
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -125,21 +125,23 @@ class Analyzer(
object CT
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14568
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/14568
**[Test build #63472 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63472/consoleFull)**
for PR 14568 at commit
[`8198d9c`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14531
**[Test build #63457 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63457/consoleFull)**
for PR 14531 at commit
[`6180e80`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14531
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14531
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63457/
Test PASSed.
---
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/14397
@dongjoon-hyun I'll take a look in the morning (CET)
---
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
Github user rajeshbalamohan commented on the issue:
https://github.com/apache/spark/pull/14537
Thanks @mallman . Fixed review comments in latest commit.
---
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 dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14397
Thank you so much! Then, see you later. :)
---
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 fe
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14568
**[Test build #63473 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63473/consoleFull)**
for PR 14568 at commit
[`b713f23`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14537
**[Test build #63474 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63474/consoleFull)**
for PR 14537 at commit
[`75bca1b`](https://github.com/apache/spark/commit/7
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r74155283
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -358,6 +376,119 @@ private[mesos] trait MesosSchedu
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14547
**[Test build #63447 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63447/consoleFull)**
for PR 14547 at commit
[`fe256f7`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14547
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/14547
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63447/
Test FAILed.
---
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r74155413
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -358,6 +376,119 @@ private[mesos] trait MesosSchedu
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14568
**[Test build #63466 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63466/consoleFull)**
for PR 14568 at commit
[`81e342d`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14568
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/14568
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63466/
Test FAILed.
---
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r74156410
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -358,6 +376,119 @@ private[mesos] trait MesosSchedu
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r74156582
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -358,6 +376,119 @@ private[mesos] trait MesosSchedu
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14547
**[Test build #3210 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3210/consoleFull)**
for PR 14547 at commit
[`fe256f7`](https://github.com/apache/spark/commit/
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/14547
CC: @hhbyyh Would you mind taking a look at this since you're familiar with
GBTs? Thanks in advance! This should be one of the most important
improvements in terms of accuracy, especially once w
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r74156824
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -358,6 +376,119 @@ private[mesos] trait MesosSchedu
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14568
**[Test build #63469 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63469/consoleFull)**
for PR 14568 at commit
[`f78d6aa`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14568
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63469/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14568
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 skonto commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r74157953
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -358,6 +376,119 @@ private[mesos] trait MesosSchedule
Github user skonto commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r74158089
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -358,6 +376,119 @@ private[mesos] trait MesosSchedule
Github user skonto commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r74158079
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -358,6 +376,119 @@ private[mesos] trait MesosSchedule
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14568
**[Test build #63475 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63475/consoleFull)**
for PR 14568 at commit
[`4d0cd3a`](https://github.com/apache/spark/commit/4
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/14524
+1 for changing this. If left as is, JIRAs like this one will be a
recurring theme. A lot of users would probably not even notice, then wind up
getting degraded results and not know why.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14552
**[Test build #63460 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63460/consoleFull)**
for PR 14552 at commit
[`880a1f4`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14552
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/14552
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63460/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14569
**[Test build #63461 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63461/consoleFull)**
for PR 14569 at commit
[`1b17aff`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14569
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/14569
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63461/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14472
**[Test build #63476 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63476/consoleFull)**
for PR 14472 at commit
[`5eb01fe`](https://github.com/apache/spark/commit/5
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14560
(BTW, it seems this is what hive does)
---
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/14570
**[Test build #63463 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63463/consoleFull)**
for PR 14570 at commit
[`0cf8f43`](https://github.com/apache/spark/commit/
Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/14524#discussion_r74159065
--- Diff: python/pyspark/ml/param/shared.py ---
@@ -311,7 +311,7 @@ class HasSeed(Params):
def __init__(self):
super(HasSeed,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14570
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/14570
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63463/
Test PASSed.
---
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r74159170
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -358,6 +376,119 @@ private[mesos] trait MesosSchedu
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14568
**[Test build #63470 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63470/consoleFull)**
for PR 14568 at commit
[`1107182`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14568
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/14568
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63470/
Test FAILed.
---
GitHub user dongjoon-hyun opened a pull request:
https://github.com/apache/spark/pull/14571
[SPARK-16983][SQL] Add `prettyName` for row_number, dense_rank,
percent_rank, cume_dist
## What changes were proposed in this pull request?
Currently, two-word window functions like
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14571
**[Test build #63477 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63477/consoleFull)**
for PR 14571 at commit
[`2c875dc`](https://github.com/apache/spark/commit/2
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14527
**[Test build #63468 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63468/consoleFull)**
for PR 14527 at commit
[`fdeb02e`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14527
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/14527
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63468/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14511
**[Test build #63464 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63464/consoleFull)**
for PR 14511 at commit
[`81f2b1c`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14511
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/14511
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63464/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14512
**[Test build #63478 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63478/consoleFull)**
for PR 14512 at commit
[`8ab05c6`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14475
**[Test build #63465 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63465/consoleFull)**
for PR 14475 at commit
[`6ce70a9`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14475
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/14475
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63465/
Test PASSed.
---
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/14572
[SPARK-16552] [FOLLOW-UP] [SQL] Store the Inferred Schemas into External
Catalog Tables when Creating Tables
### What changes were proposed in this pull request?
Address the comments by @yhu
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/13152#discussion_r74161348
--- Diff:
core/src/main/scala/org/apache/spark/storage/BlockManagerMasterEndpoint.scala
---
@@ -126,6 +138,10 @@ class BlockManagerMasterEndpoint(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14572
**[Test build #63479 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63479/consoleFull)**
for PR 14572 at commit
[`b7c0994`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13680
**[Test build #63467 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63467/consoleFull)**
for PR 13680 at commit
[`d9a269d`](https://github.com/apache/spark/commit/
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/13152#discussion_r74161515
--- Diff:
core/src/main/scala/org/apache/spark/storage/BlockReplicationPrioritization.scala
---
@@ -0,0 +1,78 @@
+/*
+ * Licensed to the Apache Softwa
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63467/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
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 ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/13152#discussion_r74161774
--- Diff: core/src/main/scala/org/apache/spark/storage/TopologyMapper.scala
---
@@ -0,0 +1,82 @@
+/*
+ * Licensed to the Apache Software Foundation (AS
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/13152#discussion_r74161782
--- Diff: core/src/main/scala/org/apache/spark/storage/TopologyMapper.scala
---
@@ -0,0 +1,82 @@
+/*
+ * Licensed to the Apache Software Foundation (AS
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/13152#discussion_r74161892
--- Diff: core/src/main/scala/org/apache/spark/storage/TopologyMapper.scala
---
@@ -0,0 +1,82 @@
+/*
+ * Licensed to the Apache Software Foundation (AS
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/13152#discussion_r74161947
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala
---
@@ -1088,109 +1108,88 @@ private[spark] class BlockManager(
}
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/13152#discussion_r74161980
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala
---
@@ -1088,109 +1108,88 @@ private[spark] class BlockManager(
}
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/13152#discussion_r74161937
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala
---
@@ -1088,109 +1108,88 @@ private[spark] class BlockManager(
}
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/13152#discussion_r74161999
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala
---
@@ -1088,109 +1108,88 @@ private[spark] class BlockManager(
}
Github user ericl commented on the issue:
https://github.com/apache/spark/pull/13152
LGTM. Just style comments
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/14472
**[Test build #63471 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63471/consoleFull)**
for PR 14472 at commit
[`0359756`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14472
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63471/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14472
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/14568
**[Test build #63473 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63473/consoleFull)**
for PR 14568 at commit
[`b713f23`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14568
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63473/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14568
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/14537
**[Test build #63474 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63474/consoleFull)**
for PR 14537 at commit
[`75bca1b`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14537
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/14537
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63474/
Test PASSed.
---
Github user junyangq commented on a diff in the pull request:
https://github.com/apache/spark/pull/14258#discussion_r74164731
--- Diff: R/pkg/R/install.R ---
@@ -0,0 +1,230 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/14258#discussion_r74164995
--- Diff: R/pkg/R/install.R ---
@@ -0,0 +1,230 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license
401 - 500 of 768 matches
Mail list logo