Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17330
**[Test build #75654 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75654/testReport)**
for PR 17330 at commit
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17575#discussion_r110662125
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/fpm/PrefixSpan.scala
---
@@ -232,6 +200,68 @@ class PrefixSpan private (
object PrefixSpan
Github user Syrux commented on a diff in the pull request:
https://github.com/apache/spark/pull/17575#discussion_r110662506
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/fpm/PrefixSpan.scala
---
@@ -232,6 +200,68 @@ class PrefixSpan private (
object PrefixSpan extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17527
**[Test build #75657 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75657/testReport)**
for PR 17527 at commit
Github user Syrux commented on a diff in the pull request:
https://github.com/apache/spark/pull/17575#discussion_r110664282
--- Diff:
mllib/src/test/scala/org/apache/spark/mllib/fpm/PrefixSpanSuite.scala ---
@@ -360,6 +360,55 @@ class PrefixSpanSuite extends SparkFunSuite with
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17591
**[Test build #75660 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75660/testReport)**
for PR 17591 at commit
Github user Syrux commented on a diff in the pull request:
https://github.com/apache/spark/pull/17575#discussion_r110667171
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/fpm/PrefixSpan.scala
---
@@ -232,6 +200,68 @@ class PrefixSpan private (
object PrefixSpan extends
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17330#discussion_r110671091
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/subquery.scala
---
@@ -236,6 +244,12 @@ case class ScalarSubquery(
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17575#discussion_r110671100
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/fpm/PrefixSpan.scala
---
@@ -232,6 +200,68 @@ class PrefixSpan private (
object PrefixSpan
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17330#discussion_r110671334
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/subquery.scala
---
@@ -59,6 +58,13 @@ abstract class SubqueryExpression(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17591
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/17591
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75660/
Test PASSed.
---
Github user Syrux commented on a diff in the pull request:
https://github.com/apache/spark/pull/17575#discussion_r110669561
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/fpm/PrefixSpan.scala
---
@@ -232,6 +200,68 @@ class PrefixSpan private (
object PrefixSpan extends
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17587
@kiszk The plan may not be very helpful, as the only difference in the plan
is `MapObjects.lambdaFunction`, before it was just `LambdaVariable`, but now
it's `Cast(LambdaVariable, xxx)`
---
If
Github user Syrux commented on a diff in the pull request:
https://github.com/apache/spark/pull/17575#discussion_r110671916
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/fpm/PrefixSpan.scala
---
@@ -232,6 +200,68 @@ class PrefixSpan private (
object PrefixSpan extends
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17330#discussion_r110671813
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/subquery.scala
---
@@ -59,6 +58,13 @@ abstract class SubqueryExpression(
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17589
@wgtmac and @sameeragarwal, I am sorry it took me a while to reopen this. I
was doing this for vectorized reader together but I realised that the PR is
getting too big.
I kind of had
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17575#discussion_r110600408
--- Diff:
mllib/src/test/scala/org/apache/spark/mllib/fpm/PrefixSpanSuite.scala ---
@@ -360,6 +360,55 @@ class PrefixSpanSuite extends SparkFunSuite with
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17575#discussion_r110600152
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/fpm/PrefixSpan.scala
---
@@ -232,6 +200,68 @@ class PrefixSpan private (
object PrefixSpan
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17575#discussion_r110600704
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/fpm/PrefixSpan.scala
---
@@ -232,6 +200,68 @@ class PrefixSpan private (
object PrefixSpan
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17589
Probably it is too late for 2.2. If anyone feels so, I will close this for
now again.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17575#discussion_r110600533
--- Diff:
mllib/src/test/scala/org/apache/spark/mllib/fpm/PrefixSpanSuite.scala ---
@@ -360,6 +360,55 @@ class PrefixSpanSuite extends SparkFunSuite with
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17575#discussion_r110599784
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/fpm/PrefixSpan.scala
---
@@ -232,6 +200,68 @@ class PrefixSpan private (
object PrefixSpan
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/17556
Just a flaky test. Can't be related
---
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 MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/16774#discussion_r110606671
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tuning/ValidatorParams.scala ---
@@ -67,6 +71,39 @@ private[ml] trait ValidatorParams extends HasSeed
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/16774#discussion_r110605376
--- Diff: docs/ml-tuning.md ---
@@ -55,6 +55,9 @@ for multiclass problems. The default metric used to
choose the best `ParamMap` c
method in each of
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/16774#discussion_r110610974
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/tuning/CrossValidatorSuite.scala ---
@@ -121,6 +121,33 @@ class CrossValidatorSuite
}
}
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/17480#discussion_r110802758
--- Diff:
core/src/main/scala/org/apache/spark/ExecutorAllocationManager.scala ---
@@ -249,7 +249,14 @@ private[spark] class ExecutorAllocationManager(
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/17563
Does the PR nobody deal with it?@srowen
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17601
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/17601
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75680/
Test PASSed.
---
Github user witgo commented on a diff in the pull request:
https://github.com/apache/spark/pull/17480#discussion_r110804557
--- Diff:
core/src/main/scala/org/apache/spark/ExecutorAllocationManager.scala ---
@@ -249,7 +249,14 @@ private[spark] class ExecutorAllocationManager(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17599
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/16781
**[Test build #75686 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75686/testReport)**
for PR 16781 at commit
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17602#discussion_r110810961
--- Diff: python/pyspark/sql/streaming.py ---
@@ -405,8 +405,8 @@ def json(self, path, schema=None,
primitivesAsString=None, prefersDecimal=None,
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17602#discussion_r110810827
--- Diff: python/pyspark/sql/readwriter.py ---
@@ -634,7 +634,9 @@ def saveAsTable(self, name, format=None, mode=None,
partitionBy=None, **options)
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17602#discussion_r110811551
--- Diff: docs/sql-programming-guide.md ---
@@ -883,7 +883,7 @@ Configuration of Parquet can be done using the
`setConf` method on `SparkSession
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17596#discussion_r110811597
--- Diff:
core/src/main/scala/org/apache/spark/util/InternalLongAccumulator.scala ---
@@ -0,0 +1,50 @@
+/*
+ * Licensed to the Apache Software
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17602
**[Test build #75690 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75690/testReport)**
for PR 17602 at commit
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17602#discussion_r110814635
--- Diff: docs/sql-programming-guide.md ---
@@ -897,7 +897,7 @@ For a regular multi-line JSON file, set the `wholeFile`
option to `true`.
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17602#discussion_r110814617
--- Diff: docs/sql-programming-guide.md ---
@@ -883,7 +883,7 @@ Configuration of Parquet can be done using the
`setConf` method on `SparkSession
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17436
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/17436
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75685/
Test PASSed.
---
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/17480#discussion_r110804952
--- Diff:
core/src/main/scala/org/apache/spark/ExecutorAllocationManager.scala ---
@@ -249,7 +249,14 @@ private[spark] class ExecutorAllocationManager(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17436
**[Test build #75685 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75685/testReport)**
for PR 17436 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17600
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75679/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17600
**[Test build #75679 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75679/testReport)**
for PR 17600 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17600
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/17599
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75678/
Test PASSed.
---
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/17582
@tgravescs sorry for the confuse.
>if base URL's ACL (spark.acls.enable) is enabled but user A has no view
permission. User "A" cannot see the app list but could still access details of
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17581
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75682/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17581
**[Test build #75682 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75682/testReport)**
for PR 17581 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17581
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 viirya commented on the issue:
https://github.com/apache/spark/pull/17491
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 so,
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17596#discussion_r110810746
--- Diff:
core/src/main/scala/org/apache/spark/util/InternalLongAccumulator.scala ---
@@ -0,0 +1,50 @@
+/*
+ * Licensed to the Apache Software
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17602#discussion_r110810554
--- Diff: python/pyspark/sql/readwriter.py ---
@@ -173,8 +173,8 @@ def json(self, path, schema=None,
primitivesAsString=None, prefersDecimal=None,
Github user dbtsai commented on the issue:
https://github.com/apache/spark/pull/17601
Merged into 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 wishes so,
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17602#discussion_r110811804
--- Diff: python/pyspark/sql/readwriter.py ---
@@ -634,7 +634,9 @@ def saveAsTable(self, name, format=None, mode=None,
partitionBy=None, **options)
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17602#discussion_r110812638
--- Diff: docs/sql-programming-guide.md ---
@@ -897,7 +897,7 @@ For a regular multi-line JSON file, set the `wholeFile`
option to `true`.
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/17581#discussion_r110813819
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -359,6 +359,16 @@ object SQLConf {
.booleanConf
Github user shaolinliu commented on the issue:
https://github.com/apache/spark/pull/17581
Ok, I have modified the description.
---
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/17436
**[Test build #75685 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75685/testReport)**
for PR 17436 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17495
**[Test build #75684 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75684/testReport)**
for PR 17495 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17495
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/17495
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75684/
Test PASSed.
---
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/17599
LGTM pending Jenkins.
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17601
**[Test build #75680 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75680/consoleFull)**
for PR 17601 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17491
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75683/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17491
**[Test build #75683 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75683/testReport)**
for PR 17491 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17491
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17491
**[Test build #75687 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75687/testReport)**
for PR 17491 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17599
---
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 opened a pull request:
https://github.com/apache/spark/pull/17602
[MINOR][DOCS] JSON APIs related documentation fixes
## What changes were proposed in this pull request?
This PR proposes corrections related to JSON APIs, including rendering
links in
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17602#discussion_r110810364
--- Diff: docs/sql-programming-guide.md ---
@@ -883,7 +883,7 @@ Configuration of Parquet can be done using the
`setConf` method on `SparkSession
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17602#discussion_r110811091
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameReader.scala ---
@@ -268,8 +268,8 @@ class DataFrameReader private[sql](sparkSession:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17602
**[Test build #75691 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75691/testReport)**
for PR 17602 at commit
Github user dbtsai commented on the issue:
https://github.com/apache/spark/pull/17600
Merged into mater.
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17602
**[Test build #75692 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75692/testReport)**
for PR 17602 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17596
@mridulm I actually have the same plan, I think it's an overkill to
implement TaskMetrics with accumulators, we don't need to merge the accumulator
updates at driver side for TaskMetrics
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17540
**[Test build #75681 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75681/testReport)**
for PR 17540 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17540
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/17540
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75681/
Test FAILed.
---
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17480#discussion_r110803588
--- Diff:
core/src/main/scala/org/apache/spark/ExecutorAllocationManager.scala ---
@@ -249,7 +249,14 @@ private[spark] class ExecutorAllocationManager(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17599
**[Test build #75678 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75678/testReport)**
for PR 17599 at commit
Github user facaiy commented on the issue:
https://github.com/apache/spark/pull/17556
@srowen Hi, I forget unit tests in python and R. Where can I find document
about creating develop environment? thanks.
---
If your project is set up for it, you can reply to this email and have
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/17599
Merging in master/branch-2.1.
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/16677
**[Test build #75688 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75688/testReport)**
for PR 16677 at commit
Github user mridulm commented on the issue:
https://github.com/apache/spark/pull/17596
My comments were based on a fix in 1.6; actually lot of values were
actually observed to be 0 for a lot of cases - just a few were not (even here
it is relevant - resultSize, gctime, various bytes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17495
**[Test build #75684 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75684/testReport)**
for PR 17495 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17602
**[Test build #75689 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75689/testReport)**
for PR 17602 at commit
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17602#discussion_r110810413
--- Diff: docs/sql-programming-guide.md ---
@@ -897,7 +897,7 @@ For a regular multi-line JSON file, set the `wholeFile`
option to `true`.
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/17480#discussion_r110797585
--- Diff:
core/src/main/scala/org/apache/spark/ExecutorAllocationManager.scala ---
@@ -249,7 +249,14 @@ private[spark] class ExecutorAllocationManager(
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/17593
sorting can change?I do not think so.
Even if the sorting is only show the last 200, and I raised the issue is
not contradictory.
The last 200 are the concept of a batch of data.
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/17587
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 viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17587#discussion_r110676717
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/ScalaReflection.scala
---
@@ -270,12 +270,13 @@ object ScalaReflection extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17592
**[Test build #75661 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75661/testReport)**
for PR 17592 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17592
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75661/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17592
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/17591
**[Test build #75663 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75663/testReport)**
for PR 17591 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17591
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
301 - 400 of 402 matches
Mail list logo