Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23012
**[Test build #98848 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98848/testReport)**
for PR 23012 at commit
[`f153413`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23012
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/23016
**[Test build #98853 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98853/testReport)**
for PR 23016 at commit
[`5c4949d`](https://github.com/apache/spark/commit/5c
Github user j-baker commented on the issue:
https://github.com/apache/spark/pull/22946
have got a test and have hopefully fixed the existing ones!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Fo
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22946
**[Test build #98852 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98852/testReport)**
for PR 22946 at commit
[`417582d`](https://github.com/apache/spark/commit/41
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22946
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/22946
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98851/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22946
**[Test build #98851 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98851/testReport)**
for PR 22946 at commit
[`0a558c4`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22911
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/22911
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98838/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22946
**[Test build #98851 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98851/testReport)**
for PR 22946 at commit
[`0a558c4`](https://github.com/apache/spark/commit/0a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23038
**[Test build #98850 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98850/testReport)**
for PR 23038 at commit
[`c53ca48`](https://github.com/apache/spark/commit/c5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23009
**[Test build #4425 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4425/testReport)**
for PR 23009 at commit
[`c8424af`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22911
**[Test build #98838 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98838/testReport)**
for PR 22911 at commit
[`d4a84bc`](https://github.com/apache/spark/commit/d
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20788#discussion_r233678942
--- Diff: python/pyspark/sql/tests/test_dataframe.py ---
@@ -375,6 +375,19 @@ def test_generic_hints(self):
plan = df1.join(df2.hint("broadc
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20788
Thanks. @DylanGuedes.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: rev
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22612
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/22612
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98837/
Test PASSed.
---
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22967
Thank you, @shaneknapp . Like the current job
`spark-master-test-maven-hadoop-2.7-ubuntu-scala-2.12/`, `master` branch will
need it. So,
from `spark-master-test-maven-hadoop-2.7-ubuntu-s
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22612
**[Test build #98837 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98837/testReport)**
for PR 22612 at commit
[`8f20857`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23038
**[Test build #98849 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98849/testReport)**
for PR 23038 at commit
[`b7a47c2`](https://github.com/apache/spark/commit/b7
Github user shahidki31 commented on the issue:
https://github.com/apache/spark/pull/23016
Thanks @srowen , I will update the PR
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comman
Github user shahidki31 commented on the issue:
https://github.com/apache/spark/pull/23038
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23012
**[Test build #98848 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98848/testReport)**
for PR 23012 at commit
[`f153413`](https://github.com/apache/spark/commit/f1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23012
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/5038/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23012
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/23014
---
-
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/22946
**[Test build #98847 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98847/testReport)**
for PR 22946 at commit
[`b6a191a`](https://github.com/apache/spark/commit/b6
Github user shaneknapp commented on the issue:
https://github.com/apache/spark/pull/22967
@dongjoon-hyun not a problem, i'll need to update the build config(s)
what branches will need 2.12 vs 2.11?
---
-
To uns
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23014
Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user ifilonenko commented on the issue:
https://github.com/apache/spark/pull/23037
I see this customization to be specific towards how you build your custom
Docker image. Unless it is relevant towards testing, we are trying to keep the
default Docker image as lightweight as pos
Github user shahidki31 commented on the issue:
https://github.com/apache/spark/pull/23038
I'll update the PR, for test failures
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comman
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22967
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23039
@MaxGekk, I think the main purpose of this PR is rather to introduce
`spark.sql.debug.maxToStringFields` .. let's fix PR description and title for
that.
---
--
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23038
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98840/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23038
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/22967
Merged to master. Thank you all!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23038
**[Test build #98840 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98840/testReport)**
for PR 23038 at commit
[`ed98958`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23034
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/5037/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23034
Merged build finished. Test PASSed.
---
-
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/22967
Hi, @shaneknapp.
If we make Scala 2.11 as our default, can we reuse the following Job for
Scala 2.11 instead?
-
https://amplab.cs.berkeley.edu/jenkins/view/Spark%20QA%20Test%20(Dashboa
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22911
**[Test build #98846 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98846/testReport)**
for PR 22911 at commit
[`78b76a8`](https://github.com/apache/spark/commit/78
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23033
**[Test build #98845 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98845/testReport)**
for PR 23033 at commit
[`79bb522`](https://github.com/apache/spark/commit/79
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23034
**[Test build #98844 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98844/testReport)**
for PR 23034 at commit
[`aee074f`](https://github.com/apache/spark/commit/ae
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23033
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/23033
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/5035/
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23012
Yup, will address the other comments and update the PR accordingly.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23034
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/5034/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23034
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/23016
I'm not sure about that, because the returned PrefixSpanModel has an RDD
that depends on that RDD. We could cache the final RDD instead and materialize
it; that could make more sense.
In ot
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22911#discussion_r233667509
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/KerberosConfDriverFeatureStep.scala
---
@@ -69,97 +72,191 @@
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23034
**[Test build #98843 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98843/testReport)**
for PR 23034 at commit
[`950eed9`](https://github.com/apache/spark/commit/95
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23034
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/23033
Yup will do.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...
Github user shahidki31 commented on the issue:
https://github.com/apache/spark/pull/23016
cc @srowen Kindly review
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user shahidki31 commented on the issue:
https://github.com/apache/spark/pull/23038
cc @vanzin @srowen kindly review
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e
Github user ifilonenko commented on a diff in the pull request:
https://github.com/apache/spark/pull/22911#discussion_r233660132
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/KerberosConfDriverFeatureStep.scala
---
@@ -69,97 +72,19
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22911#discussion_r233658128
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/KerberosConfDriverFeatureStep.scala
---
@@ -69,97 +72,191 @@
Github user ifilonenko commented on a diff in the pull request:
https://github.com/apache/spark/pull/22911#discussion_r233657016
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/KerberosConfDriverFeatureStep.scala
---
@@ -69,97 +72,19
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22911#discussion_r233654819
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/KerberosConfDriverFeatureStep.scala
---
@@ -69,97 +72,191 @@
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22911
**[Test build #98842 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98842/testReport)**
for PR 22911 at commit
[`7e9b3a2`](https://github.com/apache/spark/commit/7e
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22911#discussion_r233653814
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/KerberosConfDriverFeatureStep.scala
---
@@ -69,97 +72,191 @@
Github user ifilonenko commented on a diff in the pull request:
https://github.com/apache/spark/pull/22911#discussion_r233652949
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/KerberosConfDriverFeatureStep.scala
---
@@ -69,97 +72,19
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22911#discussion_r233651011
--- Diff:
resource-managers/kubernetes/core/src/test/scala/org/apache/spark/deploy/k8s/features/KerberosConfDriverFeatureStepSuite.scala
---
@@ -0,0 +1,164
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22911#discussion_r233650735
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/submit/KubernetesDriverBuilder.scala
---
@@ -81,14 +81,14 @@ private[
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22911#discussion_r233650398
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/KerberosConfDriverFeatureStep.scala
---
@@ -69,97 +72,191 @@
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23036
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98836/
Test PASSed.
---
Github user ifilonenko commented on a diff in the pull request:
https://github.com/apache/spark/pull/22911#discussion_r233650115
--- Diff:
resource-managers/kubernetes/core/src/test/scala/org/apache/spark/deploy/k8s/features/KerberosConfDriverFeatureStepSuite.scala
---
@@ -0,0 +1,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23036
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/23036
**[Test build #98836 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98836/testReport)**
for PR 23036 at commit
[`785a423`](https://github.com/apache/spark/commit/7
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22911#discussion_r233649612
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/KerberosConfDriverFeatureStep.scala
---
@@ -69,97 +72,191 @@
Github user ifilonenko commented on a diff in the pull request:
https://github.com/apache/spark/pull/22911#discussion_r233649498
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/submit/KubernetesDriverBuilder.scala
---
@@ -81,14 +81,14 @@ priv
Github user ifilonenko commented on a diff in the pull request:
https://github.com/apache/spark/pull/22911#discussion_r233649110
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/KerberosConfDriverFeatureStep.scala
---
@@ -69,97 +72,19
Github user ifilonenko commented on a diff in the pull request:
https://github.com/apache/spark/pull/22911#discussion_r233648073
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/KerberosConfDriverFeatureStep.scala
---
@@ -69,97 +72,19
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23024
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98834/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23024
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/23024
**[Test build #98834 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98834/testReport)**
for PR 23024 at commit
[`55aebe6`](https://github.com/apache/spark/commit/5
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22911#discussion_r233643587
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/BasicExecutorFeatureStep.scala
---
@@ -144,6 +144,10 @@ priv
Github user ifilonenko commented on a diff in the pull request:
https://github.com/apache/spark/pull/22911#discussion_r233642625
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/BasicExecutorFeatureStep.scala
---
@@ -144,6 +144,10 @@
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/23031
All entries are already captured in the `ConfigEntry` object, so are all
these changes really needed?
You could warn if anybody tries to update something that's in there, but
not in `SQLConf
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22598#discussion_r233630560
--- Diff:
core/src/test/scala/org/apache/spark/deploy/security/KafkaTokenUtilSuite.scala
---
@@ -0,0 +1,150 @@
+/*
+ * Licensed to the Apache Softwa
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23024
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98835/
Test PASSed.
---
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22598#discussion_r233629662
--- Diff:
core/src/test/scala/org/apache/spark/deploy/security/KafkaTokenUtilSuite.scala
---
@@ -0,0 +1,150 @@
+/*
+ * Licensed to the Apache Softwa
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22598#discussion_r233628422
--- Diff:
core/src/main/scala/org/apache/spark/deploy/security/KafkaTokenUtil.scala ---
@@ -0,0 +1,146 @@
+/*
+ * Licensed to the Apache Software Fou
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23024
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22598#discussion_r233629692
--- Diff:
core/src/test/scala/org/apache/spark/deploy/security/KafkaTokenUtilSuite.scala
---
@@ -0,0 +1,150 @@
+/*
+ * Licensed to the Apache Softwa
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22598#discussion_r233627918
--- Diff:
core/src/main/scala/org/apache/spark/deploy/security/KafkaTokenUtil.scala ---
@@ -0,0 +1,146 @@
+/*
+ * Licensed to the Apache Software Fou
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22598#discussion_r233627534
--- Diff:
core/src/main/scala/org/apache/spark/deploy/security/KafkaDelegationTokenProvider.scala
---
@@ -0,0 +1,57 @@
+/*
+ * Licensed to the Apach
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22598#discussion_r233637694
--- Diff:
core/src/main/scala/org/apache/spark/deploy/security/KafkaTokenUtil.scala ---
@@ -0,0 +1,146 @@
+/*
+ * Licensed to the Apache Software Fou
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22598#discussion_r233626891
--- Diff: core/pom.xml ---
@@ -408,6 +408,19 @@
provided
+
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22598#discussion_r233628054
--- Diff:
core/src/main/scala/org/apache/spark/deploy/security/KafkaTokenUtil.scala ---
@@ -0,0 +1,146 @@
+/*
+ * Licensed to the Apache Software Fou
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23024
**[Test build #98835 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98835/testReport)**
for PR 23024 at commit
[`58293b3`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23039
**[Test build #98841 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98841/testReport)**
for PR 23039 at commit
[`36de047`](https://github.com/apache/spark/commit/36
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23039
@hvanhovell @gatorsmile @HyukjinKwon @cloud-fan Could you take a look at
this PR, please.
---
-
To unsubscribe, e-mail: reviews-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23039
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/23039
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23039
The PR extracts a part of changes from #22429
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user MaxGekk opened a pull request:
https://github.com/apache/spark/pull/23039
[SPARK-26066][SQL] Moving truncatedString to sql/catalyst
## What changes were proposed in this pull request?
I propose to move truncatedString out of `core` to `sql/catalyst` because
it i
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22911
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/22911
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98833/
Test PASSed.
---
101 - 200 of 422 matches
Mail list logo