GitHub user redsanket opened a pull request:
https://github.com/apache/spark/pull/19140
[SPARK-21890] Credentials not being passed to add the tokens
I observed this while running a oozie job trying to connect to hbase via
spark.
It look like the creds are not being passed in
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19140
**[Test build #81425 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81425/testReport)**
for PR 19140 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19099
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19139
BTW, please update the PR title.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18692#discussion_r137111457
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/joins.scala
---
@@ -152,3 +152,71 @@ object EliminateOuterJoin extends
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/19139#discussion_r137115627
--- Diff: docs/sql-programming-guide.md ---
@@ -733,8 +733,9 @@ SELECT * FROM parquetTable
Table partitioning is a common optimization
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19140
**[Test build #81431 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81431/testReport)**
for PR 19140 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19124#discussion_r137120663
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveStrategies.scala ---
@@ -145,15 +146,27 @@ class DetermineTableStats(session:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18950
**[Test build #81427 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81427/testReport)**
for PR 18950 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19124#discussion_r137102745
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala
---
@@ -85,6 +88,14 @@ case class
Github user kevinyu98 commented on a diff in the pull request:
https://github.com/apache/spark/pull/12646#discussion_r137108746
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -503,69 +504,319 @@ case class
Github user kevinyu98 commented on a diff in the pull request:
https://github.com/apache/spark/pull/12646#discussion_r137108805
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -503,69 +504,319 @@ case class
Github user kevinyu98 commented on a diff in the pull request:
https://github.com/apache/spark/pull/12646#discussion_r137108881
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -503,69 +504,319 @@ case class
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/18704
ping @cloud-fan
---
-
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/18966#discussion_r137164501
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -769,16 +769,27 @@ class CodegenContext {
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19131
Jenkins, test this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19050#discussion_r137167216
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SubquerySuite.scala
---
@@ -875,4 +876,70 @@ class SubquerySuite extends QueryTest with
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19050#discussion_r137167260
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SubquerySuite.scala
---
@@ -875,4 +876,70 @@ class SubquerySuite extends QueryTest with
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19124
**[Test build #81435 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81435/testReport)**
for PR 19124 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19050
LGTM pending Jenkins
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/18935
Merging to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/18935
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19124#discussion_r137169152
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcFileFormat.scala
---
@@ -0,0 +1,42 @@
+/*
+ * Licensed to the
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19124#discussion_r137171798
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSourceStrategy.scala
---
@@ -130,10 +130,12 @@ case class
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/19124#discussion_r137173190
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSourceStrategy.scala
---
@@ -130,10 +130,12 @@ case class
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/19124#discussion_r137174463
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcFileFormat.scala
---
@@ -0,0 +1,42 @@
+/*
+ * Licensed
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/19124#discussion_r137174337
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -206,6 +206,9 @@ case class AlterTableAddColumnsCommand(
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19122#discussion_r137175343
--- Diff: python/pyspark/ml/tuning.py ---
@@ -255,18 +257,24 @@ def _fit(self, dataset):
randCol = self.uid + "_rand"
df =
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19122#discussion_r137162378
--- Diff: python/pyspark/ml/tuning.py ---
@@ -255,18 +257,24 @@ def _fit(self, dataset):
randCol = self.uid + "_rand"
df =
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19140
@redsanket can you please test this with a secure Hadoop environment using
spark-submit (not Oozie), I don't want to bring in any regression here.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17254
Build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17254
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81437/
Test FAILed.
---
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19124#discussion_r137171079
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -206,6 +206,9 @@ case class AlterTableAddColumnsCommand(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19050
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81439/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19050
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/19050
**[Test build #81439 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81439/testReport)**
for PR 19050 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19122#discussion_r137162089
--- Diff: python/pyspark/ml/tuning.py ---
@@ -255,18 +257,23 @@ def _fit(self, dataset):
randCol = self.uid + "_rand"
df =
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17254
**[Test build #81437 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81437/testReport)**
for PR 17254 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19124#discussion_r137170172
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala ---
@@ -848,4 +851,19 @@ object DDLUtils {
}
}
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19124#discussion_r137170969
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala ---
@@ -848,4 +851,19 @@ object DDLUtils {
}
}
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/19124
Oh, thank you for review, @viirya, @HyukjinKwon and @gatorsmile !
I'll follow up your comments!
---
-
To unsubscribe,
Github user jmchung commented on the issue:
https://github.com/apache/spark/pull/18865
Could @gatorsmile and @HyukjinKwon please share some instructions for
revised details on exception message? The current message indicates the reason
of disallowance when users just select the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19050
**[Test build #81439 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81439/testReport)**
for PR 19050 at commit
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19124#discussion_r137169608
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcFileFormat.scala
---
@@ -0,0 +1,42 @@
+/*
+ * Licensed to
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19124#discussion_r137169805
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcFileFormat.scala
---
@@ -0,0 +1,42 @@
+/*
+ * Licensed to the
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/19124#discussion_r137174215
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -206,6 +206,9 @@ case class AlterTableAddColumnsCommand(
Github user caneGuy commented on the issue:
https://github.com/apache/spark/pull/19132
@ajbozarth I have updated the implementation which only access FS in
FSHistoryServerProvider
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19124
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/19124
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81435/
Test PASSed.
---
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/19050
Thanks @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/18692
@cloud-fan : In event when the (set of join keys) is a superset of (child
node's partitioning keys), its possible to avoid shuffle :
https://github.com/apache/spark/pull/19054 ... this can help
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19050
**[Test build #81440 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81440/testReport)**
for PR 19050 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19124#discussion_r137170635
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala ---
@@ -848,4 +851,19 @@ object DDLUtils {
}
}
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19124#discussion_r137171539
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -206,6 +206,9 @@ case class AlterTableAddColumnsCommand(
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/19124#discussion_r137174591
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala ---
@@ -848,4 +851,19 @@ object DDLUtils {
}
}
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19136
**[Test build #81441 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81441/testReport)**
for PR 19136 at commit
Github user KevinZwx commented on the issue:
https://github.com/apache/spark/pull/18903
test this please
---
-
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/18875
**[Test build #81416 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81416/testReport)**
for PR 18875 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19056
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/18875
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81416/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19056
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81415/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19056
**[Test build #81415 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81415/testReport)**
for PR 19056 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/19139
I added `(including TEXT/CSV/JSON/ORC/Parquet)` and updated the PR title,
too.
---
-
To unsubscribe, e-mail:
Github user kevinyu98 commented on a diff in the pull request:
https://github.com/apache/spark/pull/12646#discussion_r137108608
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -503,69 +504,319 @@ case class
Github user kevinyu98 commented on a diff in the pull request:
https://github.com/apache/spark/pull/12646#discussion_r137111624
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -503,69 +504,319 @@ case class
Github user brad-kaiser commented on the issue:
https://github.com/apache/spark/pull/19041
Thanks vanzin.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/19124#discussion_r137096383
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveStrategies.scala ---
@@ -145,15 +146,27 @@ class DetermineTableStats(session:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19140
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19140
ok to test
---
-
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/19124
**[Test build #81424 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81424/testReport)**
for PR 19124 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19099
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81418/
Test PASSed.
---
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/19132
Though this may fix the problem it is 100% the wrong way to do it for
multiple reasons:
1. The UI should not touch the FS, FS access it abstracted out so the UI is
can work with multiple FS
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19099
Merged build finished. Test PASSed.
---
-
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/19139#discussion_r137101003
--- Diff: docs/sql-programming-guide.md ---
@@ -733,7 +733,7 @@ SELECT * FROM parquetTable
Table partitioning is a common optimization
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19124#discussion_r137103061
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/orc/OrcFileFormat.scala ---
@@ -83,6 +83,8 @@ class OrcFileFormat extends FileFormat with
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/19124#discussion_r137107658
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/orc/OrcFileFormat.scala ---
@@ -83,6 +83,8 @@ class OrcFileFormat extends FileFormat with
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19139
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/19124#discussion_r137122638
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveStrategies.scala ---
@@ -145,15 +146,27 @@ class DetermineTableStats(session:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19140
**[Test build #81426 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81426/testReport)**
for PR 19140 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19140
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81426/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19140
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/19099
**[Test build #81418 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81418/testReport)**
for PR 19099 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19124
That is normal. When we find a bug, it normally means we ignore it in more
than one place. Thus, we need to check all the other code paths that could
trigger it.
---
Github user janewangfb commented on a diff in the pull request:
https://github.com/apache/spark/pull/18975#discussion_r137106989
--- Diff:
sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 ---
@@ -241,11 +241,21 @@ query
: ctes? queryNoWith
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/19124#discussion_r137107152
--- Diff:
sql/core/src/test/resources/sql-tests/results/show_columns.sql.out ---
@@ -19,23 +19,23 @@ struct<>
-- !query 2
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19139#discussion_r137112679
--- Diff: docs/sql-programming-guide.md ---
@@ -733,8 +733,9 @@ SELECT * FROM parquetTable
Table partitioning is a common optimization
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19099
LGTM
---
-
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/18935
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/18935
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81420/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19124
**[Test build #81430 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81430/testReport)**
for PR 19124 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19139
**[Test build #81429 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81429/testReport)**
for PR 19139 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19139
Thanks! Merged to master/2.2
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18875
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/16774
This should be ready to merge @jkbradley @MLnick
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/19139
@gatorsmile .
For this PR, I just update the doc only. Thank you so much!
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19139
**[Test build #81423 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81423/testReport)**
for PR 19139 at commit
Github user redsanket commented on a diff in the pull request:
https://github.com/apache/spark/pull/19140#discussion_r137096611
--- Diff:
core/src/main/scala/org/apache/spark/deploy/security/HadoopFSDelegationTokenProvider.scala
---
@@ -103,15 +103,17 @@ private[deploy] class
Github user redsanket commented on the issue:
https://github.com/apache/spark/pull/19140
Previous discussion on this PR is here
https://github.com/apache/spark/pull/19103
---
-
To unsubscribe, e-mail:
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/19124
Thank you, @gatorsmile . The PR becomes much general.
When I started this PR, I didn't notice that the current Spark had so many
missing cases like this.
---
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/19138
LGTM. Thanks @brkyvz. Merging to master and 2.2
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
1 - 100 of 514 matches
Mail list logo
