Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21669#discussion_r224156599
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/HadoopConfExecutorFeatureStep.scala
---
@@ -0,0 +1,48 @@
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21669#discussion_r224161045
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/hadooputils/HadoopBootstrapUtil.scala
---
@@ -0,0 +1,283 @@
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21669#discussion_r224161661
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/hadooputils/HadoopBootstrapUtil.scala
---
@@ -0,0 +1,283 @@
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21669#discussion_r224160540
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/KerberosConfDriverFeatureStep.scala
---
@@ -0,0 +1,168 @@
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21669#discussion_r224160471
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/KerberosConfDriverFeatureStep.scala
---
@@ -0,0 +1,168 @@
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21669#discussion_r224160090
--- Diff: docs/security.md ---
@@ -722,7 +722,83 @@ with encryption, at least.
The Kerberos login will be periodically renewed using the provided
creden
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21669#discussion_r224161587
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/hadooputils/HadoopBootstrapUtil.scala
---
@@ -0,0 +1,283 @@
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21669#discussion_r224156872
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/KerberosConfDriverFeatureStep.scala
---
@@ -0,0 +1,156 @@
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21669#discussion_r224156790
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/HadoopSparkUserExecutorFeatureStep.scala
---
@@ -0,0 +1,43 @
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21669#discussion_r224160567
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/KerberosConfDriverFeatureStep.scala
---
@@ -0,0 +1,168 @@
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22686
Closing this per https://github.com/apache/spark/pull/22686
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.or
Github user gengliangwang commented on the issue:
https://github.com/apache/spark/pull/22595
I read the original PR https://github.com/apache/spark/pull/14617, and it
is wired that the two columns are hidden in the very beginning.
Also the screenshot in PR description doesn't match
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22688
+1 to deal with this as non blocker. I understand data source v2 is under
heavy development and unstable but strongly think we should backport .. it
breaks a basic operation ..
---
--
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22688#discussion_r224153758
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/sources/v2/DataSourceV2Suite.scala
---
@@ -190,12 +190,13 @@ class DataSourceV2Suite extends Query
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22688
ah good point. I think the original design of append operator assumes the
table already exists, so a schema should be provided. If we treat file path as
a table, then append should fail for your c
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22657#discussion_r224151216
--- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
@@ -106,4 +107,14 @@ abstract class SparkFunSuite
}
}
+ lazy
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22623
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/22623
Merging to master / 2.4.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22593
Although it's the Scala API, it's callable from Java just as well. There's
no Java-specific API here. So, yeah, actually it makes sense to have javadoc
and scaladoc for this. And I think there are pr
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22547
**[Test build #97206 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97206/testReport)**
for PR 22547 at commit
[`a35d98c`](https://github.com/apache/spark/commit/a3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22547
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/22547
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-unified/3854/
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22594#discussion_r224146203
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileScanRDD.scala
---
@@ -70,6 +70,8 @@ class FileScanRDD(
private
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22688
Another concern is, it doesn't sound to me straightforward that readsupport
is created and executed in write path.
So, at least, we should just read the schema when it's needed (when sav
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22641
Thank you very much @srowen @fjh100456
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional com
Github user niofire commented on the issue:
https://github.com/apache/spark/pull/22593
From
https://spark.apache.org/docs/2.3.2/api/java/org/apache/spark/sql/streaming/DataStreamWriter.html
![image](https://user-images.githubusercontent.com/2295469/46749482-b3351400-cc6a-11e8-834d
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22688
Other datasources such as Parquet, ORC and JDBC won't be adoptable to the
current design since we can't read the schema from the information given the
code I provided above.
---
-
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22641
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user tengpeng commented on the issue:
https://github.com/apache/spark/pull/21524
Let's close it for now.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revi
Github user tengpeng closed the pull request at:
https://github.com/apache/spark/pull/21524
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22641
Merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22688
The point is not write only datasource, @cloud-fan. For instance,
`spark.read.range(1).format("source").write("non-existent-path")`. There's no
way to read the schema. There's no complaints so f
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22671
I like the change. This test is down to 5 seconds now. Unfortunately I
don't see speedup in other Kafka tests, but I think we should leave this in.
---
-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22583
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/22583
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97197/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22583
**[Test build #97197 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97197/testReport)**
for PR 22583 at commit
[`e0f3160`](https://github.com/apache/spark/commit/e
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22593
Have a look at `Column.scala` and `Dataset.scala` in
`org.apache.spark.sql`. But, on a second look, this is how I see the lists
render:
https://user-images.githubusercontent.com/822522/46748
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/21816#discussion_r224128493
--- Diff:
core/src/test/scala/org/apache/spark/deploy/rest/StandaloneRestSubmitSuite.scala
---
@@ -83,6 +83,26 @@ class StandaloneRestSubmitSuite extends Sp
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22672
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22593
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/22593
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97204/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22593
**[Test build #97204 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97204/testReport)**
for PR 22593 at commit
[`f356cf3`](https://github.com/apache/spark/commit/f
Github user niofire commented on the issue:
https://github.com/apache/spark/pull/22593
@srowen Guessing you're referring to some other files? Could you link?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.ap
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22593
**[Test build #97204 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97204/testReport)**
for PR 22593 at commit
[`f356cf3`](https://github.com/apache/spark/commit/f3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22237
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97198/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22237
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22237
**[Test build #97198 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97198/testReport)**
for PR 22237 at commit
[`910f1c5`](https://github.com/apache/spark/commit/9
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/22288#discussion_r224113173
--- Diff: docs/configuration.md ---
@@ -1583,6 +1583,14 @@ Apart from these, the following properties are also
available, and may be useful
drive
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/22288#discussion_r224105069
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/BlacklistTracker.scala ---
@@ -146,21 +146,31 @@ private[scheduler] class BlacklistTracker (
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/22288#discussion_r224111068
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.scala ---
@@ -117,6 +117,10 @@ private[spark] class TaskSchedulerImpl(
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/22288#discussion_r224107221
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.scala ---
@@ -415,9 +419,61 @@ private[spark] class TaskSchedulerImpl(
Github user niofire commented on the issue:
https://github.com/apache/spark/pull/22593
Sorry, was out of town, will update
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e
Github user abellina commented on a diff in the pull request:
https://github.com/apache/spark/pull/21688#discussion_r224108070
--- Diff: core/src/main/scala/org/apache/spark/ui/jobs/StagePage.scala ---
@@ -290,17 +210,11 @@ private[ui] class StagePage(parent: StagesTab, store:
AppS
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22688
Do we have users complain about it? In the new write API design, a data
source must provide a schema.
I don't think it's practical that people need a write-only data source
which can acce
Github user abellina commented on a diff in the pull request:
https://github.com/apache/spark/pull/21688#discussion_r224106218
--- Diff:
core/src/main/scala/org/apache/spark/status/api/v1/StagesResource.scala ---
@@ -102,4 +103,124 @@ private[v1] class StagesResource extends
BaseA
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/22668#discussion_r224101829
--- Diff: core/src/main/scala/org/apache/spark/ui/PagedTable.scala ---
@@ -123,10 +123,9 @@ private[ui] trait PagedTable[T] {
/**
* Retu
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22674
**[Test build #97203 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97203/testReport)**
for PR 22674 at commit
[`3ffa536`](https://github.com/apache/spark/commit/3f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22674
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/22674
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-unified/3853/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22688
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/22688
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97196/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22657
**[Test build #97202 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97202/testReport)**
for PR 22657 at commit
[`778629d`](https://github.com/apache/spark/commit/77
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22688
**[Test build #97196 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97196/testReport)**
for PR 22688 at commit
[`a445e16`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22674
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97200/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22674
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22674
**[Test build #97200 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97200/testReport)**
for PR 22674 at commit
[`a25524b`](https://github.com/apache/spark/commit/a
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/22657#discussion_r224091433
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/util/DateTimeTestUtils.scala
---
@@ -26,6 +26,16 @@ object DateTimeTestUtils {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22674
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-unified/3852/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22674
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/22657
> Then we don't need any randomness here, just pick one timezone(like PST?)
and test it.
I took 8 out of 627. Going to do the same in the PR:
https://github.com/apache/spark/pull/22379
-
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22657#discussion_r224086763
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/util/DateTimeTestUtils.scala
---
@@ -26,6 +26,16 @@ object DateTimeTestUtils {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22674
**[Test build #97200 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97200/testReport)**
for PR 22674 at commit
[`a25524b`](https://github.com/apache/spark/commit/a2
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22657
**[Test build #97201 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97201/testReport)**
for PR 22657 at commit
[`5f8bf79`](https://github.com/apache/spark/commit/5f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22686
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97195/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22686
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22686
**[Test build #97195 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97195/testReport)**
for PR 22686 at commit
[`227920a`](https://github.com/apache/spark/commit/2
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20999#discussion_r224082261
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala ---
@@ -521,35 +521,114 @@ case class AlterTableRenamePartitionCommand(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22685
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/22685
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97194/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22687
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/22687
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97193/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22685
**[Test build #97194 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97194/testReport)**
for PR 22685 at commit
[`52ed24e`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22687
**[Test build #97193 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97193/testReport)**
for PR 22687 at commit
[`0f43db6`](https://github.com/apache/spark/commit/0
Github user skonto commented on a diff in the pull request:
https://github.com/apache/spark/pull/22381#discussion_r224071384
--- Diff: core/src/main/scala/org/apache/spark/status/AppStatusSource.scala
---
@@ -0,0 +1,81 @@
+/*
+ * Licensed to the Apache Software Foundation (
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22674#discussion_r224071116
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -3356,21 +3356,11 @@ class Dataset[T] private[sql](
* user-registered c
Github user skonto commented on a diff in the pull request:
https://github.com/apache/spark/pull/22381#discussion_r224071170
--- Diff:
core/src/main/scala/org/apache/spark/status/AppStatusListener.scala ---
@@ -280,6 +284,11 @@ private[spark] class AppStatusListener(
private
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22684
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-unified/3851/
Github user lbialy commented on the issue:
https://github.com/apache/spark/pull/22309
Hello. I've just found this PR while trying to debug serialization errors
for a case class containing value classes. This would be very helpful!
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22684
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22684
**[Test build #97199 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97199/testReport)**
for PR 22684 at commit
[`c2eb87a`](https://github.com/apache/spark/commit/c2
Github user skonto commented on the issue:
https://github.com/apache/spark/pull/22381
Thanx a lot @aditanase ! I will update the PR.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional c
Github user skonto commented on a diff in the pull request:
https://github.com/apache/spark/pull/22381#discussion_r224070656
--- Diff: core/src/main/scala/org/apache/spark/status/AppStatusSource.scala
---
@@ -0,0 +1,81 @@
+/*
+ * Licensed to the Apache Software Foundation (
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22598
Merged build finished. Test PASSed.
---
-
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/22237#discussion_r224069683
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/FailureSafeParser.scala
---
@@ -15,50 +15,57 @@
* limitations under the Li
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22598
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97188/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22598
**[Test build #97188 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97188/testReport)**
for PR 22598 at commit
[`fffd139`](https://github.com/apache/spark/commit/f
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22682
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22682
thanks, merging to master!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22684
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97187/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22684
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
301 - 400 of 501 matches
Mail list logo