Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18350
Can one of the admins verify this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18343#discussion_r122625325
--- Diff:
core/src/main/scala/org/apache/spark/serializer/KryoSerializer.scala ---
@@ -175,6 +175,7 @@ class KryoSerializer(conf: SparkConf)
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18343#discussion_r122625202
--- Diff: core/src/main/scala/org/apache/spark/scheduler/MapStatus.scala ---
@@ -141,7 +141,7 @@ private[spark] class HighlyCompressedMapStatus private (
Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/18350
Hi @srowen, would you mind take a look?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18343
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
GitHub user ConeyLiu opened a pull request:
https://github.com/apache/spark/pull/18350
[MINOR] Fix some typo of the document
## What changes were proposed in this pull request?
Fix some typo of the document.
## How was this patch tested?
Existing tests.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18343
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/78239/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18343
**[Test build #78239 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78239/testReport)**
for PR 18343 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17328
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17328
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/78242/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17328
**[Test build #78242 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78242/testReport)**
for PR 17328 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18343#discussion_r122623575
--- Diff: core/src/main/scala/org/apache/spark/scheduler/MapStatus.scala ---
@@ -141,7 +141,7 @@ private[spark] class HighlyCompressedMapStatus private (
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18343#discussion_r122623298
--- Diff: core/src/main/scala/org/apache/spark/scheduler/MapStatus.scala ---
@@ -141,7 +141,7 @@ private[spark] class HighlyCompressedMapStatus private (
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18346
cc @cloud-fan @gatorsmile
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18269
let's wait @mallman 's response to make sure this patch does fix the problem
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18269#discussion_r122622206
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/InMemoryFileIndex.scala
---
@@ -248,60 +245,94 @@ object InMemoryFileIndex
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18269#discussion_r122622157
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/InMemoryFileIndex.scala
---
@@ -248,60 +245,94 @@ object InMemoryFileIndex
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18269#discussion_r122622031
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/InMemoryFileIndex.scala
---
@@ -248,60 +245,94 @@ object InMemoryFileIndex
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17471
I was just looking though PRs for my curiosity. Please let me leave a
gentle ping @sitalkedia.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18269#discussion_r122621941
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/InMemoryFileIndex.scala
---
@@ -248,60 +245,94 @@ object InMemoryFileIndex
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/18303
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18092
**[Test build #78247 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78247/testReport)**
for PR 18092 at commit
Github user liyichao commented on a diff in the pull request:
https://github.com/apache/spark/pull/18092#discussion_r122621671
--- Diff:
core/src/test/scala/org/apache/spark/storage/BlockManagerSuite.scala ---
@@ -1281,6 +1286,61 @@ class BlockManagerSuite extends SparkFunSuite
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18343#discussion_r122621682
--- Diff: core/src/main/scala/org/apache/spark/scheduler/MapStatus.scala ---
@@ -141,7 +141,7 @@ private[spark] class HighlyCompressedMapStatus private (
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18303
Thanks! Merging to master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18303
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/78235/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18346
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18303
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18346
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18346
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/78238/
Test PASSed.
---
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18290
LGTM, only one question: are we going to support it? To reuse the same
shuffle service across workers or to allow multiple shuffle services on one
host?
---
If your project is set up for it,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18346
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/78237/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18303
**[Test build #78235 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78235/testReport)**
for PR 18303 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18346
**[Test build #78238 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78238/testReport)**
for PR 18346 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18346
**[Test build #78237 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78237/testReport)**
for PR 18346 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18346
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/78236/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18346
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18346
**[Test build #78236 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78236/testReport)**
for PR 18346 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18092#discussion_r122620995
--- Diff:
core/src/test/scala/org/apache/spark/storage/BlockManagerSuite.scala ---
@@ -1281,6 +1286,61 @@ class BlockManagerSuite extends SparkFunSuite
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18343#discussion_r122620862
--- Diff: core/src/main/scala/org/apache/spark/scheduler/MapStatus.scala ---
@@ -141,7 +141,7 @@ private[spark] class HighlyCompressedMapStatus private (
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17401
gentle ping @ash211. I just wonder if it is active now.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/18324#discussion_r122620720
--- Diff: python/pyspark/worker.py ---
@@ -177,8 +180,11 @@ def process():
process()
except Exception:
try:
+
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15417
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15417
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/78243/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15417
**[Test build #78243 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78243/testReport)**
for PR 15417 at commit
Github user liyichao commented on a diff in the pull request:
https://github.com/apache/spark/pull/18092#discussion_r122620600
--- Diff:
core/src/test/scala/org/apache/spark/storage/BlockManagerSuite.scala ---
@@ -1281,6 +1286,59 @@ class BlockManagerSuite extends SparkFunSuite
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/17758
ok, I'll recheck.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18343
@wangyum Can you also add a test for this?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18323
**[Test build #78245 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78245/testReport)**
for PR 18323 at commit
Github user zenglinxi0615 commented on a diff in the pull request:
https://github.com/apache/spark/pull/14085#discussion_r122620464
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/commands.scala ---
@@ -113,8 +113,9 @@ case class AddFile(path: String) extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18092
**[Test build #78246 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78246/testReport)**
for PR 18092 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18092
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17758
shall we check duplicated columns in write path?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17395
hmmm. @uncleGen, shell we close this for now? reopening when it's ready
would welcome.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17328
+1 for this PR.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so,
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18324#discussion_r122620265
--- Diff: python/pyspark/worker.py ---
@@ -177,8 +180,11 @@ def process():
process()
except Exception:
try:
+
Github user liyichao commented on a diff in the pull request:
https://github.com/apache/spark/pull/18092#discussion_r122620196
--- Diff:
core/src/test/scala/org/apache/spark/storage/BlockManagerSuite.scala ---
@@ -1281,6 +1286,59 @@ class BlockManagerSuite extends SparkFunSuite
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15417
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15417
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/78241/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15417
**[Test build #78241 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78241/testReport)**
for PR 15417 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17084
gentle ping @imatiach-msft .
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/78244/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18025
**[Test build #78244 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78244/testReport)**
for PR 18025 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18343
@wangyum Thanks for updating. Can you try to disable kyro and try it again?
So we can verify it.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17681
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/78240/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17681
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16766#discussion_r122619579
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -2603,12 +2603,27 @@ class Dataset[T] private[sql](
* current
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16766#discussion_r122619526
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -2603,12 +2603,27 @@ class Dataset[T] private[sql](
* current
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17681
**[Test build #78240 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78240/testReport)**
for PR 17681 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18343
Because we write/read `hugeBlockSizes` in `writeExternal`/`readExternal`,
it seems to me that it is intended to be serialized. So I think removing
`transient` should be ok.
LGTM cc
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18349
Can one of the admins verify this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user ZiyueHuang commented on the issue:
https://github.com/apache/spark/pull/18349
@zsxwing Could you please review this PR?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/18343
@viirya Yes, I' using `org.apache.spark.serializer.KryoSerializer`, [master
branch](https://github.com/apache/spark/tree/ce49428ef7d640c1734e91ffcddc49dbc8547ba7)
still has this issue, error logs:
GitHub user ZiyueHuang opened a pull request:
https://github.com/apache/spark/pull/18349
[SPARK-20927][SS] Change some operators in Dataset to no-op for a streaming
query.
## What changes were proposed in this pull request?
Change some operators(persist, unpersist,
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16347
gentle ping @junegunn on ^.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18343
I think this should be addressed before 2.2. I already asked notice of
other committers on dev mailing list.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14957
@xuanyuanking, let's close this and help review #16578 if you agree on the
comments above.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18343
@wangyum Are you using kyro serializer? I think it is why you hit this
issue.
Once you use kyro, I think the `readExternal` in
`HighlyCompressedMapStatus` won't be used to deserialize the
Github user actuaryzhang commented on the issue:
https://github.com/apache/spark/pull/18025
This is how the doc for column_aggregate_functions looks like (only
snapshot of the main parts):
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18303
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18092
LGTM except 2 minor comments
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18092#discussion_r122617592
--- Diff:
core/src/test/scala/org/apache/spark/storage/BlockManagerSuite.scala ---
@@ -1281,6 +1286,59 @@ class BlockManagerSuite extends SparkFunSuite
Github user actuaryzhang commented on a diff in the pull request:
https://github.com/apache/spark/pull/18025#discussion_r122617531
--- Diff: R/pkg/R/stats.R ---
@@ -52,22 +52,17 @@ setMethod("crosstab",
collect(dataFrame(sct))
})
-#'
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18025
**[Test build #78244 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78244/testReport)**
for PR 18025 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18092#discussion_r122617448
--- Diff:
core/src/test/scala/org/apache/spark/storage/BlockManagerSuite.scala ---
@@ -1281,6 +1286,59 @@ class BlockManagerSuite extends SparkFunSuite
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/13893
ya, this still exists. Let me find some time to resolve this.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user actuaryzhang commented on a diff in the pull request:
https://github.com/apache/spark/pull/18025#discussion_r122617405
--- Diff: R/pkg/R/stats.R ---
@@ -52,22 +52,17 @@ setMethod("crosstab",
collect(dataFrame(sct))
})
-#'
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/12257
gentle ping @sarutak on ^
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user jaceklaskowski commented on a diff in the pull request:
https://github.com/apache/spark/pull/18347#discussion_r122617147
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -465,6 +465,8 @@ case class DataSource(
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18296
Actually this is a bug fix, and it's a small fix(without tests, only 3
lines), so backporting in to 2.2
---
If your project is set up for it, you can reply to this email and have your
reply
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/11887
@skonto, do you maybe know (or a wild guess) when we would be able to
proceed this? Probably, closing this for now and reopening might be an option.
---
If your project is set up for it, you
Github user jaceklaskowski commented on a diff in the pull request:
https://github.com/apache/spark/pull/18347#discussion_r122616876
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -465,6 +465,8 @@ case class DataSource(
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/11420
Let's close this if it is not in a progress.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user actuaryzhang commented on a diff in the pull request:
https://github.com/apache/spark/pull/18025#discussion_r122616787
--- Diff: R/pkg/R/stats.R ---
@@ -52,22 +52,17 @@ setMethod("crosstab",
collect(dataFrame(sct))
})
-#'
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/18296
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/11205
gentle ping @rustgi, have you maybe had some time to confirm this patch
maybe? It sounds the only thing we need here is the confirmation.
---
If your project is set up for it, you can reply to
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18296
LGTM, merging to master!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/10861
Hi @rajeshbalamohan, I think this should be a mergeable state at least and
the conflicts and style issues should be resolved. Would you be able to update
this for now?
---
If your project is
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18334
+1 to provide a flag to automatically trigger the stats updates. We cat set
it false by default to not surprise users
---
If your project is set up for it, you can reply to this email and have
1 - 100 of 288 matches
Mail list logo
