GitHub user shenh062326 opened a pull request:
https://github.com/apache/spark/pull/14557
[SPARK-16709][CORE] Kill the running task if stage failed
## What changes were proposed in this pull request?
At SPARK-16709, when a stage failed, but the running task is still
runnin
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14556
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 feat
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14555#discussion_r74006226
--- Diff:
mllib-local/src/main/scala/org/apache/spark/ml/linalg/Vectors.scala ---
@@ -560,11 +554,25 @@ class SparseVector @Since("2.0.0") (
@Since("
Github user zjffdu commented on a diff in the pull request:
https://github.com/apache/spark/pull/14555#discussion_r74006057
--- Diff:
mllib-local/src/main/scala/org/apache/spark/ml/linalg/Vectors.scala ---
@@ -560,11 +554,25 @@ class SparseVector @Since("2.0.0") (
@Since("
GitHub user Sherry302 opened a pull request:
https://github.com/apache/spark/pull/14556
[SPARK-16966][Core] Make App Name to the valid name instead of a randâ¦
## What changes were proposed in this pull request?
In the SparkSession, before setting "spark.app.name" to
"java.util
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14551#discussion_r74005725
--- Diff: core/src/test/scala/org/apache/spark/util/UtilsSuite.scala ---
@@ -874,4 +874,38 @@ class UtilsSuite extends SparkFunSuite with
ResetSystemPropertie
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14551#discussion_r74005704
--- Diff: core/src/test/scala/org/apache/spark/util/UtilsSuite.scala ---
@@ -874,4 +874,38 @@ class UtilsSuite extends SparkFunSuite with
ResetSystemPropertie
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14551#discussion_r74005614
--- Diff: core/src/test/scala/org/apache/spark/util/UtilsSuite.scala ---
@@ -874,4 +874,38 @@ class UtilsSuite extends SparkFunSuite with
ResetSystemPropertie
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14551#discussion_r74005685
--- Diff: core/src/test/scala/org/apache/spark/util/UtilsSuite.scala ---
@@ -874,4 +874,38 @@ class UtilsSuite extends SparkFunSuite with
ResetSystemPropertie
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14554
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
e
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14551#discussion_r74005625
--- Diff: core/src/test/scala/org/apache/spark/util/UtilsSuite.scala ---
@@ -874,4 +874,38 @@ class UtilsSuite extends SparkFunSuite with
ResetSystemPropertie
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14554
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63415/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14554
**[Test build #63415 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63415/consoleFull)**
for PR 14554 at commit
[`d9ba88f`](https://github.com/apache/spark/commit/
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14555#discussion_r74005289
--- Diff:
mllib-local/src/main/scala/org/apache/spark/ml/linalg/Vectors.scala ---
@@ -560,11 +554,25 @@ class SparseVector @Since("2.0.0") (
@Since("
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14555#discussion_r74005178
--- Diff:
mllib-local/src/main/scala/org/apache/spark/ml/linalg/Vectors.scala ---
@@ -560,11 +554,25 @@ class SparseVector @Since("2.0.0") (
@Since("
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14555#discussion_r74005184
--- Diff:
mllib-local/src/main/scala/org/apache/spark/ml/linalg/Vectors.scala ---
@@ -560,11 +554,25 @@ class SparseVector @Since("2.0.0") (
@Since("
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13701
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13701
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63412/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13701
**[Test build #63412 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63412/consoleFull)**
for PR 13701 at commit
[`cee74b7`](https://github.com/apache/spark/commit/
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14531#discussion_r74004679
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -80,13 +83,49 @@ case class CreateTableLikeCommand(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14531#discussion_r74004636
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -80,13 +83,49 @@ case class CreateTableLikeCommand(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14531
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14531
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63413/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14531
**[Test build #63413 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63413/consoleFull)**
for PR 14531 at commit
[`b820be8`](https://github.com/apache/spark/commit/
Github user mallman commented on the issue:
https://github.com/apache/spark/pull/14537
@rajeshbalamohan, the changes to `HiveMetastoreCatalog.scala` look
reasonable. This mirrors the behavior of this method before the `if
(fileType.equals("parquet"))` expression was introduced in
1e8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13701
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63410/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13701
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13701
**[Test build #63410 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63410/consoleFull)**
for PR 13701 at commit
[`0b38ba1`](https://github.com/apache/spark/commit/
Github user mallman commented on a diff in the pull request:
https://github.com/apache/spark/pull/14537#discussion_r74003788
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -294,7 +294,9 @@ private[hive] class HiveMetastoreCatalog(spark
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/14520
@sethah
Thanks for your careful review!
The PR here already passing the bcFeaturesStd and bcCoeffs as constructor
args to the `LogisticAggregator`, like your PR #14109
You mean
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14546
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14546
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63408/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14546
**[Test build #63408 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63408/consoleFull)**
for PR 14546 at commit
[`7c3d732`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63420 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63420/consoleFull)**
for PR 13146 at commit
[`8119f6d`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14546
**[Test build #63419 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63419/consoleFull)**
for PR 14546 at commit
[`1dc193a`](https://github.com/apache/spark/commit/1
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14531#discussion_r74001311
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -80,13 +83,49 @@ case class CreateTableLikeCommand(
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14531#discussion_r74001182
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -80,13 +83,49 @@ case class CreateTableLikeCommand(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14546
**[Test build #63418 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63418/consoleFull)**
for PR 14546 at commit
[`1ca8d59`](https://github.com/apache/spark/commit/1
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14554#discussion_r74000577
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/PartitioningUtils.scala
---
@@ -41,22 +42,23 @@ object PartitionDirectory {
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14554#discussion_r74000543
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/udaf.scala ---
@@ -266,7 +266,7 @@ private[sql] class MutableAggregationBufferIm
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14554#discussion_r74000537
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/udaf.scala ---
@@ -204,7 +204,7 @@ sealed trait BufferSetterGetterUtils {
/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14555
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14555
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63416/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14555
**[Test build #63416 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63416/consoleFull)**
for PR 14555 at commit
[`57b9c84`](https://github.com/apache/spark/commit/
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14531#discussion_r74000488
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -80,13 +83,49 @@ case class CreateTableLikeCommand(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14554#discussion_r74000462
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkPlanInfo.scala ---
@@ -47,7 +47,7 @@ class SparkPlanInfo(
}
}
-
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63417 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63417/consoleFull)**
for PR 13146 at commit
[`3826f33`](https://github.com/apache/spark/commit/3
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14531#discussion_r74000396
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -80,13 +83,49 @@ case class CreateTableLikeCommand(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14555
**[Test build #63416 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63416/consoleFull)**
for PR 14555 at commit
[`57b9c84`](https://github.com/apache/spark/commit/5
GitHub user zjffdu opened a pull request:
https://github.com/apache/spark/pull/14555
[SPARK-16965][MLLIB][PYSPARK] Fix bound checking for SparseVector.
## What changes were proposed in this pull request?
1. In scala, add negative low bound checking and put all the low/upper
Github user yhuai commented on the issue:
https://github.com/apache/spark/pull/14492
That's a good point. Let me close 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
enabl
Github user yhuai closed the pull request at:
https://github.com/apache/spark/pull/14492
---
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 enab
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/14546#discussion_r73999422
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1252,7 +1252,9 @@ class Analyzer(
c
Github user NarineK commented on a diff in the pull request:
https://github.com/apache/spark/pull/14384#discussion_r73999377
--- Diff: R/pkg/R/mllib.R ---
@@ -632,3 +642,147 @@ setMethod("predict", signature(object =
"AFTSurvivalRegressionModel"),
function(object, ne
Github user NarineK commented on a diff in the pull request:
https://github.com/apache/spark/pull/14384#discussion_r73999041
--- Diff: R/pkg/R/mllib.R ---
@@ -632,3 +642,147 @@ setMethod("predict", signature(object =
"AFTSurvivalRegressionModel"),
function(object, ne
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/14546#discussion_r73998920
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1252,7 +1252,9 @@ class Analyzer(
c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14517
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
e
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/13775
ping @hvanhovell @rxin @liancheng @yhuai Could you review this now? Thanks.
---
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 pro
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14517
**[Test build #63414 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63414/consoleFull)**
for PR 14517 at commit
[`96df186`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14517
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63414/
Test FAILed.
---
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14546#discussion_r73998693
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1252,7 +1252,9 @@ class Analyzer(
case
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/14222
ping @rxin any thoughts on this? It is waiting for a while. Thanks.
---
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 doe
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14517
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14517
**[Test build #63411 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63411/consoleFull)**
for PR 14517 at commit
[`0bc078a`](https://github.com/apache/spark/commit/
Github user NarineK commented on a diff in the pull request:
https://github.com/apache/spark/pull/14384#discussion_r73998522
--- Diff: R/pkg/R/mllib.R ---
@@ -632,3 +642,147 @@ setMethod("predict", signature(object =
"AFTSurvivalRegressionModel"),
function(object, ne
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14517
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63411/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14554
**[Test build #63415 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63415/consoleFull)**
for PR 14554 at commit
[`d9ba88f`](https://github.com/apache/spark/commit/d
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/14452
ping @cloud-fan @hvanhovell @liancheng Could you review this change? Thanks.
---
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 pr
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/14554
cc @cloud-fan
---
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 th
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14531#discussion_r73998018
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -80,13 +83,49 @@ case class CreateTableLikeCommand(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14531#discussion_r73997983
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -80,13 +83,49 @@ case class CreateTableLikeCommand(
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14546#discussion_r73997535
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1252,7 +1252,9 @@ class Analyzer(
case
Github user ScrapCodes commented on the issue:
https://github.com/apache/spark/pull/14087
@tdas Ping !
---
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 i
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14102
@yhuai Could you please take another 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 feat
Github user ScrapCodes commented on the issue:
https://github.com/apache/spark/pull/14151
@rxin Ping !
---
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 i
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63406/
Test PASSed.
---
Github user gatorsmile closed the pull request at:
https://github.com/apache/spark/pull/13907
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14531
**[Test build #63413 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63413/consoleFull)**
for PR 14531 at commit
[`b820be8`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14517
**[Test build #63414 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63414/consoleFull)**
for PR 14517 at commit
[`96df186`](https://github.com/apache/spark/commit/9
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/13907
I see. Let me close it.
---
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 wish
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63406 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63406/consoleFull)**
for PR 13146 at commit
[`8e1600e`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14547
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63407/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14547
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14547
**[Test build #63407 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63407/consoleFull)**
for PR 14547 at commit
[`cfaee0f`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14517
**[Test build #63411 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63411/consoleFull)**
for PR 14517 at commit
[`0bc078a`](https://github.com/apache/spark/commit/0
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13701
**[Test build #63412 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63412/consoleFull)**
for PR 13701 at commit
[`cee74b7`](https://github.com/apache/spark/commit/c
Github user GregBowyer commented on the issue:
https://github.com/apache/spark/pull/14517
Amended commit with style changes from MLNick. Can someone call the OK to
test please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as we
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14489
LGTM. @felixcheung feel free to merge this and let me know if the commit
scripts work fine
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14554
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63409/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14554
**[Test build #63409 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63409/consoleFull)**
for PR 14554 at commit
[`78d1381`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14554
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14065
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14065
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63405/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14065
**[Test build #63405 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63405/consoleFull)**
for PR 14065 at commit
[`71df428`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13701
**[Test build #63410 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63410/consoleFull)**
for PR 13701 at commit
[`0b38ba1`](https://github.com/apache/spark/commit/0
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14554
**[Test build #63409 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63409/consoleFull)**
for PR 14554 at commit
[`78d1381`](https://github.com/apache/spark/commit/7
GitHub user rxin opened a pull request:
https://github.com/apache/spark/pull/14554
[SPARK-16964][SQL] Remove private[sql] and private[spark] from
sql.execution package
## What changes were proposed in this pull request?
This package is meant to be internal, and as a result it do
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14546
**[Test build #63408 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63408/consoleFull)**
for PR 14546 at commit
[`7c3d732`](https://github.com/apache/spark/commit/7
Github user lw-lin commented on a diff in the pull request:
https://github.com/apache/spark/pull/14298#discussion_r73995159
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/PercentileApprox.scala
---
@@ -0,0 +1,462 @@
+/*
+ * Licens
1 - 100 of 629 matches
Mail list logo
