Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18424
Now we will have this for row group filtering in most cases after #15049. I
believe it makes sense in this case.
---
-
To un
Github user umesh9794 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17631#discussion_r150809485
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcUtils.scala
---
@@ -121,7 +121,8 @@ object JdbcUtils extends Loggin
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18424
**[Test build #83844 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83844/testReport)**
for PR 18424 at commit
[`8c443a7`](https://github.com/apache/spark/commit/8c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18424
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83844/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18424
**[Test build #83844 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83844/testReport)**
for PR 18424 at commit
[`8c443a7`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18424
Build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18424
I believe I need to cc @jiangxb1987 and @viirya too who activiely reviewed
my PR.
---
-
To unsubscribe, e-mail: reviews-unsu
Github user xuanyuanking commented on the issue:
https://github.com/apache/spark/pull/17702
@zsxwing Thanks for your comments, ready for review.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18424
cc @liancheng too who I know is insightful in this.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For ad
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19691
**[Test build #83839 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83839/testReport)**
for PR 19691 at commit
[`9832ec5`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19743
**[Test build #83841 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83841/testReport)**
for PR 19743 at commit
[`cc9ecc6`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19743
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/19691
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/19691
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83839/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19743
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83841/
Test PASSed.
---
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/19640
It looks like this:
https://user-images.githubusercontent.com/5399861/32779430-be47bac8-c978-11e7-97d1-fd3e3db9166f.png";>
---
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19640
why it has `PM` postfix at the end?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19439
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19439
FWIW, I have no more comments and seems fine. I happened to review this and
manually test multiple times.
---
-
To unsubscri
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19439
**[Test build #83845 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83845/testReport)**
for PR 19439 at commit
[`c4a6dfe`](https://github.com/apache/spark/commit/c4
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/19640
Generated by the `toLocaleTimeString ()` method:
---
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19740#discussion_r150825051
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/vectorized/ColumnVectorSuite.scala
---
@@ -179,7 +179,7 @@ class ColumnVectorSuite extends S
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/18853
retest this please.
---
-
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/18853
**[Test build #83847 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83847/testReport)**
for PR 18853 at commit
[`22d0355`](https://github.com/apache/spark/commit/22
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18906
**[Test build #83846 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83846/testReport)**
for PR 18906 at commit
[`9856be6`](https://github.com/apache/spark/commit/98
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/19741
cc @jerryshao
---
-
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/19720
**[Test build #83843 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83843/testReport)**
for PR 19720 at commit
[`3a5c683`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19720
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83843/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19720
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/19640
sorry, I mean we should follow the format we use in completed, started...
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spa
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18906
> Satisfying a use case that I described in my initial comments.
I am actually not quite clear of the usecases. Providing actual codes and
elaborating it should be helpful.
> C
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/18641#discussion_r150839052
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/DataFrameSuite.scala
---
@@ -2158,4 +2158,21 @@ class DataFrameSuite extends QueryTest with
SharedSQLC
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17436#discussion_r150839810
--- Diff:
core/src/main/scala/org/apache/spark/internal/config/package.scala ---
@@ -80,6 +80,11 @@ package object config {
.bytesConf(ByteUnit.M
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17436#discussion_r150840015
--- Diff: core/src/main/scala/org/apache/spark/memory/MemoryManager.scala
---
@@ -195,7 +196,7 @@ private[spark] abstract class MemoryManager(
*/
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17436#discussion_r150840518
--- Diff:
core/src/test/scala/org/apache/spark/memory/StaticMemoryManagerSuite.scala ---
@@ -43,12 +43,13 @@ class StaticMemoryManagerSuite extends
Memor
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19640
**[Test build #83848 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83848/testReport)**
for PR 19640 at commit
[`4a78965`](https://github.com/apache/spark/commit/4a
Github user wangyum commented on a diff in the pull request:
https://github.com/apache/spark/pull/19640#discussion_r150841579
--- Diff:
core/src/test/scala/org/apache/spark/deploy/history/HistoryServerSuite.scala ---
@@ -352,7 +352,7 @@ class HistoryServerSuite extends SparkFunSuit
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/18424
Please rebase this PR to the latest master. Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For ad
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/19743
cc @cloud-fan @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revi
Github user wangyum commented on a diff in the pull request:
https://github.com/apache/spark/pull/19640#discussion_r150842384
--- Diff: core/src/main/resources/org/apache/spark/ui/static/utils.js ---
@@ -46,3 +46,25 @@ function formatBytes(bytes, type) {
var i = Math.floor(
Github user wangyum commented on a diff in the pull request:
https://github.com/apache/spark/pull/19640#discussion_r150842538
--- Diff: core/src/main/resources/org/apache/spark/ui/static/historypage.js
---
@@ -37,11 +37,6 @@ function makeIdNumeric(id) {
return resl;
}
GitHub user xuanyuanking opened a pull request:
https://github.com/apache/spark/pull/19745
[SPARK-2926][Core][Follow Up] Sort shuffle reader for Spark 2.x
## What changes were proposed in this pull request?
As comment in
[SPARK-2926][https://issues.apache.org/jira/browse/SP
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/18641#discussion_r150845392
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/conditionalExpressions.scala
---
@@ -262,37 +262,91 @@ case class CaseWhenCod
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17436#discussion_r150845467
--- Diff:
core/src/test/scala/org/apache/spark/memory/StaticMemoryManagerSuite.scala ---
@@ -43,12 +44,14 @@ class StaticMemoryManagerSuite extends
Memor
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17436#discussion_r150845551
--- Diff:
core/src/test/scala/org/apache/spark/memory/UnifiedMemoryManagerSuite.scala ---
@@ -40,10 +42,12 @@ class UnifiedMemoryManagerSuite extends
Mem
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17436#discussion_r150846694
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFileFormat.scala
---
@@ -351,11 +351,12 @@ class ParquetFileF
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19745
**[Test build #83849 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83849/testReport)**
for PR 19745 at commit
[`ca43f1b`](https://github.com/apache/spark/commit/ca
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17436#discussion_r150847402
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/AggregateHashMap.java
---
@@ -40,7 +42,7 @@
*/
public class Aggrega
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17436#discussion_r150848938
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryTableScanExec.scala
---
@@ -60,9 +62,15 @@ case class InMemoryTableScan
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19640#discussion_r150849659
--- Diff: core/src/main/resources/org/apache/spark/ui/static/utils.js ---
@@ -46,3 +46,25 @@ function formatBytes(bytes, type) {
var i = Math.floo
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/17436#discussion_r150852190
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/AggregateHashMap.java
---
@@ -40,7 +42,7 @@
*/
public class AggregateHa
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17436
after some more thoughts, I think we should not rely on
`spark.memory.offHeap.enabled`. The reason is that: 1) `OffHeapColumnVector`
may have performance regression, users may want off-heap memory
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18641#discussion_r150856256
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/conditionalExpressions.scala
---
@@ -262,37 +262,91 @@ case class CaseWhe
Github user ptkool commented on the issue:
https://github.com/apache/spark/pull/18906
Here are the similar changes in the Scala API:
https://github.com/apache/spark/pull/17911
---
-
To unsubscribe, e-mail: reviews-u
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/17436
Good idea. We will introduce the new config independent of
`spark.memory.offHeap.enabled`.
---
-
To unsubscribe, e-mail: reviews-
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/17436#discussion_r150859426
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryTableScanExec.scala
---
@@ -60,9 +62,15 @@ case class InMemoryTableScanExec
GitHub user MrBago opened a pull request:
https://github.com/apache/spark/pull/19746
[SPARK-22346][ML]
## What changes were proposed in this pull request?
A new VectorSizeHint transformer was added. This transformer is meant to be
used as a pipeline stage ahead of VectorAss
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19746
**[Test build #83850 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83850/testReport)**
for PR 19746 at commit
[`df990ed`](https://github.com/apache/spark/commit/df
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19746
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83850/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19746
**[Test build #83850 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83850/testReport)**
for PR 19746 at commit
[`df990ed`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19746
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18906
I meant the actual equivalent endpoints and actual codes with usecases.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spa
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/18641#discussion_r150863818
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/conditionalExpressions.scala
---
@@ -262,37 +262,91 @@ case class CaseWhenCod
Github user wangyum commented on a diff in the pull request:
https://github.com/apache/spark/pull/19640#discussion_r150863877
--- Diff: core/src/main/resources/org/apache/spark/ui/static/utils.js ---
@@ -46,3 +46,25 @@ function formatBytes(bytes, type) {
var i = Math.floor(
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/19740
`ColumnVectorArray` or `ArrayInColumnVector` both good to me.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
F
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/19720
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/19635
@gatorsmile any further comment on this?
---
-
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/19640#discussion_r150868951
--- Diff: core/src/main/resources/org/apache/spark/ui/static/utils.js ---
@@ -46,3 +46,25 @@ function formatBytes(bytes, type) {
var i = Math.floo
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19640#discussion_r150869426
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/HistoryPage.scala ---
@@ -56,6 +56,10 @@ private[history] class HistoryPage(parent:
History
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19640#discussion_r150869830
--- Diff: core/src/main/resources/org/apache/spark/ui/static/utils.js ---
@@ -46,3 +46,25 @@ function formatBytes(bytes, type) {
var i = Math.floo
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19479
thanks, merging to master!
---
-
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/19638
**[Test build #3986 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3986/testReport)**
for PR 19638 at commit
[`68e467e`](https://github.com/apache/spark/commit/6
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19479
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19638
**[Test build #3986 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3986/testReport)**
for PR 19638 at commit
[`68e467e`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18906
**[Test build #83846 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83846/testReport)**
for PR 18906 at commit
[`9856be6`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18906
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83846/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18906
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19720#discussion_r150875807
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/nullExpressions.scala
---
@@ -72,14 +72,10 @@ case class Coalesce(children: S
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19439
**[Test build #83845 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83845/testReport)**
for PR 19439 at commit
[`c4a6dfe`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19439
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83845/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19439
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19720#discussion_r150877087
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/nullExpressions.scala
---
@@ -72,14 +72,10 @@ case class Coalesce(children
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18853
**[Test build #83847 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83847/testReport)**
for PR 18853 at commit
[`22d0355`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18853
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83847/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18853
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user mukunku commented on the issue:
https://github.com/apache/spark/pull/1737
I'm using spark 2.2.0 and still have this issue:
Illegal Parquet type: FIXED_LEN_BYTE_ARRAY
---
-
To unsubscribe, e-mail: revi
Github user liyinan926 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19468#discussion_r150881619
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/scheduler/cluster/k8s/KubernetesClusterSchedulerBackend.scala
---
@@ -0,0 +1
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19468
**[Test build #83851 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83851/testReport)**
for PR 19468 at commit
[`0ab9ca7`](https://github.com/apache/spark/commit/0a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19468
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/19468
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83851/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19468
**[Test build #83851 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83851/testReport)**
for PR 19468 at commit
[`0ab9ca7`](https://github.com/apache/spark/commit/0
Github user squito commented on the issue:
https://github.com/apache/spark/pull/19698
I merged this to master, since its holding up a bunch more of the history
server stuff, but I'd encourage other reviews to take a look still @cloud-fan
@ajbozarth @jerryshao
---
-
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/17436#discussion_r150889105
--- Diff:
core/src/test/scala/org/apache/spark/memory/StaticMemoryManagerSuite.scala ---
@@ -43,12 +44,14 @@ class StaticMemoryManagerSuite extends
MemoryMan
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19698
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/17436#discussion_r150889421
--- Diff:
core/src/test/scala/org/apache/spark/memory/UnifiedMemoryManagerSuite.scala ---
@@ -40,10 +42,12 @@ class UnifiedMemoryManagerSuite extends
MemoryM
Github user squito commented on the issue:
https://github.com/apache/spark/pull/19681
@vanzin merge conflict since I merged the Job & stage page change
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.o
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19640
**[Test build #83848 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83848/testReport)**
for PR 19640 at commit
[`4a78965`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19640
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83848/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19640
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/19726
Thank you for review, @gatorsmile and @ueshin .
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For addi
101 - 200 of 414 matches
Mail list logo
