Github user huaxingao commented on a diff in the pull request:
https://github.com/apache/spark/pull/21513#discussion_r194247139
--- Diff: python/pyspark/ml/clustering.py ---
@@ -1156,6 +1157,213 @@ def getKeepLastCheckpoint(self):
return
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21506
**[Test build #91627 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91627/testReport)**
for PR 21506 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21506
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91627/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21506
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21258
**[Test build #91630 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91630/testReport)**
for PR 21258 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21520
**[Test build #91631 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91631/testReport)**
for PR 21520 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21523
**[Test build #91632 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91632/testReport)**
for PR 21523 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21513
**[Test build #91628 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91628/testReport)**
for PR 21513 at commit
GitHub user mgaido91 opened a pull request:
https://github.com/apache/spark/pull/21523
[SPARK-24506][UI] Add UI filters also to thriftserver tab
## What changes were proposed in this pull request?
Currently, `spark.ui.filters` are not applied to the pages added in the
Github user huaxingao commented on a diff in the pull request:
https://github.com/apache/spark/pull/21513#discussion_r194247137
--- Diff: python/pyspark/ml/clustering.py ---
@@ -1156,6 +1157,213 @@ def getKeepLastCheckpoint(self):
return
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21520
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21520
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/3884/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21061
**[Test build #91629 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91629/testReport)**
for PR 21061 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21258
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/3883/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21258
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21523
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/3885/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21523
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21513
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/3881/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21513
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/21502
@rxin this can reduce the problem since the destroy operation is invoked
when the execution of the query is terminated (in STS), instead of some time
later when the cleaner thread does it. In this
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21513
**[Test build #91628 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91628/testReport)**
for PR 21513 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21513
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21513
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91628/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21061
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21061
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/3882/
Github user daroo closed the pull request at:
https://github.com/apache/spark/pull/19789
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user jaceklaskowski commented on the issue:
https://github.com/apache/spark/pull/21510
May I ask for some help merging it? /cc @srowen @holdenk @kiszk
---
-
To unsubscribe, e-mail:
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20640
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20640
ping @IgorBerman
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20272
Is this aligned with the "in cluster client"? @foxish @mccheah
---
-
To unsubscribe, e-mail:
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/21481
Since I found an plug-in for maven, I will also include a patch to add
findBugs/SpotBugs into maven in this PR.
---
-
To
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21061
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91629/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21061
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21061
**[Test build #91629 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91629/testReport)**
for PR 21061 at commit
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21520
**[Test build #91631 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91631/testReport)**
for PR 21520 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21520
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91631/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21520
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21501
**[Test build #91634 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91634/testReport)**
for PR 21501 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20838
any update?
---
-
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/21501
**[Test build #91636 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91636/testReport)**
for PR 21501 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21258
**[Test build #91638 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91638/testReport)**
for PR 21258 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21452
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21438
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21438
I think filtering off `metricIds` still make sense right? @gatorsmile
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21438
**[Test build #91641 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91641/testReport)**
for PR 21438 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20640
**[Test build #91639 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91639/testReport)**
for PR 20640 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21452
**[Test build #91640 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91640/testReport)**
for PR 21452 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20640
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91639/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20640
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21221
**[Test build #91642 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91642/testReport)**
for PR 21221 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21258
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91630/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21258
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21258
**[Test build #91630 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91630/testReport)**
for PR 21258 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21501
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21501
**[Test build #91635 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91635/testReport)**
for PR 21501 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21501
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91635/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21258
**[Test build #91638 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91638/testReport)**
for PR 21258 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21258
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/3887/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18853
**[Test build #91633 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91633/testReport)**
for PR 18853 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20640
**[Test build #91639 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91639/testReport)**
for PR 20640 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18853
**[Test build #91633 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91633/testReport)**
for PR 18853 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21501
**[Test build #91635 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91635/testReport)**
for PR 21501 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21258
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21501
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91637/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21501
**[Test build #91637 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91637/testReport)**
for PR 21501 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21501
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
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
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/91633/
Test PASSed.
---
Github user imatiach-msft commented on the issue:
https://github.com/apache/spark/pull/20168
@tomasatdatabricks it looks like there are some conflicts that need to be
resolved, otherwise looks good to me, can you please update the PR?
---
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/21510
LGTM with one minor comment.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21510#discussion_r194266907
--- Diff: core/src/main/scala/org/apache/spark/ui/WebUI.scala ---
@@ -101,12 +101,12 @@ private[spark] abstract class WebUI(
}
/**
-
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/21520
Thank you for a lot of works to update many places. It is very hard to
split it into several pieces.
Now, we are seeing several typical patterns in the all of changes, in
paticular by
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21501
**[Test build #91636 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91636/testReport)**
for PR 21501 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21501
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91636/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21501
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/21506#discussion_r194266060
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/HDFSBackedStateStoreProvider.scala
---
@@ -280,38 +278,49 @@
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/21506#discussion_r194266029
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/HDFSBackedStateStoreProvider.scala
---
@@ -280,38 +278,49 @@
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21258
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21258
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91638/
Test PASSed.
---
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21221
probably need to be rebased
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21221
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user koeninger commented on the issue:
https://github.com/apache/spark/pull/16006
#19431 was merged, thanks for your work. This PR should probably be closed.
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21045
**[Test build #91643 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91643/testReport)**
for PR 21045 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21045
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91643/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21045
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21438
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21438
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91641/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21524
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21524
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21524
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21370#discussion_r194275282
--- Diff: docs/configuration.md ---
@@ -456,6 +456,33 @@ Apart from these, the following properties are also
available, and may be useful
from
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21370#discussion_r194275288
--- Diff: docs/configuration.md ---
@@ -456,6 +456,33 @@ Apart from these, the following properties are also
available, and may be useful
from
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21370#discussion_r194276735
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -351,8 +354,68 @@ def show(self, n=20, truncate=True, vertical=False):
else:
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21370#discussion_r194277542
--- Diff: docs/configuration.md ---
@@ -456,6 +456,33 @@ Apart from these, the following properties are also
available, and may be useful
from
Github user edwinalu commented on the issue:
https://github.com/apache/spark/pull/21221
@squito , I'm modifying ExecutorMetrics to take in the metrics array --
this will be easier for tests where we pass in set values, and seems fine for
the actual code. It will check that the length
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/testing-k8s-prb-make-spark-distribution/3886/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21501
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91634/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21501
**[Test build #91634 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91634/testReport)**
for PR 21501 at commit
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/21520
yes, this is a great work @viirya ! It would be great if we can split it
into smaller updates. What do you think about starting doing the needed changes
in smaller PRs which focus only on specific
1 - 100 of 177 matches
Mail list logo