Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21185
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21175#discussion_r184713695
--- Diff:
core/src/main/scala/org/apache/spark/util/io/ChunkedByteBuffer.scala ---
@@ -63,10 +63,12 @@ private[spark] class ChunkedByteBuffer(var
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21165
It should be `[Spark-20087]` instead of `[Spark 20087]` in the title.
---
-
To unsubscribe, e-mail: reviews-unsubscr
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21166
+1 seems you didn't test against the latest master code or 2.3?
---
-
To unsubscribe, e-mail: reviews-unsubscr
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21165
Also, please update the PR title:
`[Spark-20087][CORE] Attach accumulators / metrics to 'TaskKilled' end
reason
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21165
I'm not against the change, but since this changes the semantics of
accumulators, we should document the changes in a migration document or
something, WDYT @cloud-fan @gatorsmile
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21165#discussion_r184438550
--- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala
---
@@ -1417,10 +1417,13 @@ class DAGScheduler(
case
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21165#discussion_r184436634
--- Diff: core/src/main/scala/org/apache/spark/TaskEndReason.scala ---
@@ -212,9 +212,15 @@ case object TaskResultLost extends TaskFailedReason
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20930
Have you applied this patch: https://github.com/apache/spark/pull/17955 ?
---
-
To unsubscribe, e-mail: reviews-unsubscr
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21088
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21088
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21079
Have you seen any performance improvement with this change?
---
-
To unsubscribe, e-mail: reviews-unsubscr
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21064
@hvanhovell seems this accidentally not get merged?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21131#discussion_r183766277
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/TaskSchedulerImplSuite.scala ---
@@ -917,4 +917,111 @@ class TaskSchedulerImplSuite extends
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21083
lgtm
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20888
Seems the test case is more flaky now...
-
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89597/console
-
https://amplab.cs.berkeley.edu/jenkins/job
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21105
QQ: Have we observed the test case being flaky before?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21085
Should we manually close this PR? @squito
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21088
Assuming the {{APP_ID}}/{{EXECUTOR_ID}} would be useful under some
meaningful scenarios, the changes looks fine
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21088#discussion_r182765201
--- Diff:
core/src/main/scala/org/apache/spark/deploy/worker/ExecutorRunner.scala ---
@@ -142,7 +142,14 @@ private[deploy] class ExecutorRunner
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21084
Does it really takes a significant time to serialize the
`taskMemoryManager` ?
---
-
To unsubscribe, e-mail: reviews
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20998
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21074#discussion_r182295312
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -176,10 +176,16 @@ object TypeCoercion
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21085
LGTM!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21019
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21019
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20888
Overall this LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21019#discussion_r181778477
--- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala
---
@@ -1433,6 +1426,16 @@ class DAGScheduler
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21019#discussion_r181779186
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/DAGSchedulerSuite.scala ---
@@ -2146,6 +2146,57 @@ class DAGSchedulerSuite extends
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20888
also cc @ala @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20888#discussion_r181745760
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameRangeSuite.scala ---
@@ -152,39 +154,54 @@ class DataFrameRangeSuite extends QueryTest
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20888#discussion_r181756636
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameRangeSuite.scala ---
@@ -156,43 +156,52 @@ class DataFrameRangeSuite extends QueryTest
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21074
cc @HyukjinKwon @gatorsmile @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user jiangxb1987 opened a pull request:
https://github.com/apache/spark/pull/21074
[SPARK-21811][SQL] Fix the inconsistency behavior when finding the widest
common type
## What changes were proposed in this pull request?
Currently we find the wider common type
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21036
@guoxiaolongzte Have you tried the config
`spark.hadoopRDD.ignoreEmptySplits` ?
---
-
To unsubscribe, e-mail: reviews
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21059
I won't recommend to call `client.requestTotalExecutors` every time we try
to add executors, instead we should find out the root cause why
`client.requestTotalExecutors` didn't work as expected
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21059
@sadhen Is it possible to verify whether this still happens on our current
master?
---
-
To unsubscribe, e-mail: reviews
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19868#discussion_r180962121
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/TableReader.scala ---
@@ -176,12 +176,13 @@ class HadoopTableReader(
val
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19868#discussion_r180961987
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/TableReader.scala ---
@@ -176,12 +176,13 @@ class HadoopTableReader(
val
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20996#discussion_r180341726
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.scala ---
@@ -370,12 +370,10 @@ private[spark] class TaskSchedulerImpl
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20977
Also cc @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20913
@mshtelma You can close this PR and open a new one instead.
---
-
To unsubscribe, e-mail: reviews-unsubscr
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20993
They're in the JIRA task description
Xianjin YE <notificati...@github.com>äº2018å¹´4æ6æ¥
å¨äºä¸å8:58åéï¼
> @jiangxb1987 <https://github.com/jian
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20993
cc @cloud-fan @advancedxy @dongjoon-hyun PTAL
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
GitHub user jiangxb1987 opened a pull request:
https://github.com/apache/spark/pull/20993
[SPARK-23881][CORE][TEST] Fix flaky test
JobCancellationSuite."interruptible iterator of shuffle reader"
## What changes were proposed in this pull request?
The
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20931#discussion_r179651241
--- Diff:
core/src/main/scala/org/apache/spark/internal/io/HadoopMapReduceCommitProtocol.scala
---
@@ -186,7 +186,9 @@ class
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20931#discussion_r179651037
--- Diff:
core/src/main/scala/org/apache/spark/internal/io/HadoopMapReduceCommitProtocol.scala
---
@@ -186,7 +186,9 @@ class
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/15666
ping @mariusvniekerk
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20090
ping @liupc
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/19118
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/19528
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/19538
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20891
I'm +1 on keeping the behavior similar to that of hadoop.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20812
Should we close this then? @jinxing64 @jerryshao
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
GitHub user jiangxb1987 opened a pull request:
https://github.com/apache/spark/pull/20960
[MINOR][CORE] Show block manager id when remove RDD/Broadcast fails.
## What changes were proposed in this pull request?
Address https://github.com/apache/spark/pull/20924
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20931
cc @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20930#discussion_r178528414
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSetManager.scala ---
@@ -750,6 +752,10 @@ private[spark] class TaskSetManager
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20930#discussion_r178494006
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSetManager.scala ---
@@ -794,6 +794,19 @@ private[spark] class TaskSetManager
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20954
Thanks for reporting, I'll be looking into the flaky test.
---
-
To unsubscribe, e-mail: reviews-unsubscr
Github user jiangxb1987 closed the pull request at:
https://github.com/apache/spark/pull/20954
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20954
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
GitHub user jiangxb1987 opened a pull request:
https://github.com/apache/spark/pull/20954
[BACKPORT][SPARK-23040][CORE] Returns interruptible iterator for shuffle
reader
Backport https://github.com/apache/spark/pull/20449 and
https://github.com/apache/spark/pull/20920 to branch
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20920
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20920
Jenkins is pretty unstable recently.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20920
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20920
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20920
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20920
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20920
cc @cloud-fan @advancedxy
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e
GitHub user jiangxb1987 opened a pull request:
https://github.com/apache/spark/pull/20920
[SPARK-23040][CORE][FOLLOW-UP] Avoid double wrap result Iterator.
## What changes were proposed in this pull request?
Address https://github.com/apache/spark/pull/20449
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/19959
ping @shivaram
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/19968
ping @KaiXinXiaoLei are you still working on this?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/19881
cc @rxin
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19881#discussion_r177452303
--- Diff:
core/src/main/scala/org/apache/spark/ExecutorAllocationManager.scala ---
@@ -116,9 +120,12 @@ private[spark] class ExecutorAllocationManager
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20611
@sujith71955 Thanks for your effort on this! Is it difficult to add
(previous failing) unit tests on Spark side
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20893#discussion_r177366836
--- Diff:
core/src/main/scala/org/apache/spark/launcher/LauncherBackend.scala ---
@@ -114,10 +114,10 @@ private[spark] abstract class LauncherBackend
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20893#discussion_r177362825
--- Diff:
core/src/main/scala/org/apache/spark/launcher/LauncherBackend.scala ---
@@ -114,10 +114,10 @@ private[spark] abstract class LauncherBackend
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19881#discussion_r177362439
--- Diff:
core/src/main/scala/org/apache/spark/ExecutorAllocationManager.scala ---
@@ -116,9 +120,12 @@ private[spark] class ExecutorAllocationManager
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20893
The change looks good, cc @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20893#discussion_r176978788
--- Diff:
core/src/main/scala/org/apache/spark/launcher/LauncherBackend.scala ---
@@ -114,10 +114,10 @@ private[spark] abstract class LauncherBackend
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20895
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19881#discussion_r176328345
--- Diff:
core/src/main/scala/org/apache/spark/ExecutorAllocationManager.scala ---
@@ -116,9 +120,12 @@ private[spark] class ExecutorAllocationManager
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20853#discussion_r175662764
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
@@ -245,6 +245,17 @@ object SparkSubmit extends CommandLineUtils
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20853#discussion_r175662672
--- Diff: core/src/main/scala/org/apache/spark/deploy/DependencyUtils.scala
---
@@ -137,16 +138,32 @@ private[deploy] object DependencyUtils
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20853#discussion_r175662574
--- Diff: core/src/main/scala/org/apache/spark/deploy/DependencyUtils.scala
---
@@ -137,16 +138,32 @@ private[deploy] object DependencyUtils
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20818#discussion_r175644012
--- Diff: core/src/main/scala/org/apache/spark/ui/UIUtils.scala ---
@@ -224,6 +224,7 @@ private[spark] object UIUtils extends Logging
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20818#discussion_r175644032
--- Diff: core/src/main/scala/org/apache/spark/ui/UIUtils.scala ---
@@ -265,6 +266,7 @@ private[spark] object UIUtils extends Logging
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20812
The idea looks good, just a few comments.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20812#discussion_r174699658
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -453,8 +454,13 @@ private[spark] object Utils extends Logging
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20812#discussion_r174699762
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -752,11 +752,10 @@ private[spark] class Executor
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20812#discussion_r174699183
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -752,11 +752,10 @@ private[spark] class Executor
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20754#discussion_r174116645
--- Diff:
core/src/main/scala/org/apache/spark/ExecutorAllocationManager.scala ---
@@ -291,9 +291,6 @@ private[spark] class ExecutorAllocationManager
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20794
The fix LGTM, also cc @gengliangwang
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/19951
It's not possible to differentiate the two race conditions @vanzin
described in this code path without adding extra communication loads, since
this should be a minor issue
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20744#discussion_r174034001
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/HistoryServer.scala ---
@@ -150,14 +150,18 @@ class HistoryServer(
ui: SparkUI
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20802#discussion_r174032948
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -975,6 +975,8 @@ private[spark] object Utils extends Logging {
def
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20802#discussion_r174032702
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -975,6 +975,8 @@ private[spark] object Utils extends Logging {
def
GitHub user jiangxb1987 opened a pull request:
https://github.com/apache/spark/pull/20780
[MINOR] [SQL] [TEST] Create table using `dataSourceName` in
`HadoopFsRelationTest`
## What changes were proposed in this pull request?
This PR fixes a minor issue
Github user jiangxb1987 closed the pull request at:
https://github.com/apache/spark/pull/20768
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20649
LGTM, cc @gatorsmile @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands
301 - 400 of 1796 matches
Mail list logo