Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/14012#discussion_r70370161
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -1086,6 +1086,28 @@ object PruneFilters extends Rul
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14141
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62127/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14141
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/14141
**[Test build #62127 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62127/consoleFull)**
for PR 14141 at commit
[`5122a1a`](https://github.com/apache/spark/commit/
Github user lianhuiwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/13494#discussion_r70369134
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/OptimizeMetadataOnlyQuery.scala
---
@@ -0,0 +1,153 @@
+/*
+ * Licensed to the A
Github user lianhuiwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/13494#discussion_r70368905
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/OptimizeMetadataOnlyQuery.scala
---
@@ -0,0 +1,153 @@
+/*
+ * Licensed to the A
Github user lianhuiwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/13494#discussion_r70368861
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/OptimizeMetadataOnlyQuery.scala
---
@@ -0,0 +1,153 @@
+/*
+ * Licensed to the A
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14139
**[Test build #62136 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62136/consoleFull)**
for PR 14139 at commit
[`b98ee09`](https://github.com/apache/spark/commit/b
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/13704#discussion_r70368410
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/SimplifyCastsSuite.scala
---
@@ -0,0 +1,78 @@
+/*
+ * Licensed to the Ap
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14144
**[Test build #62135 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62135/consoleFull)**
for PR 14144 at commit
[`8b2639f`](https://github.com/apache/spark/commit/8
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/14144
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 kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/13704#discussion_r70367470
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/PrimitiveArrayBenchmark.scala
---
@@ -0,0 +1,84 @@
+/*
+ * Licensed to the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14079
**[Test build #62134 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62134/consoleFull)**
for PR 14079 at commit
[`c22aaad`](https://github.com/apache/spark/commit/c
Github user squito commented on the issue:
https://github.com/apache/spark/pull/14079
discussed this offline with @vanzin, realized that actually with the latest
design, it doesn't make sense to have so many maps inside one
`BlacklistTracker` -- a lot of info is now intended to only b
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/13894#discussion_r70367155
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tuning/CrossValidator.scala ---
@@ -56,7 +56,10 @@ private[ml] trait CrossValidatorParams extends
Valid
Github user petermaxlee commented on a diff in the pull request:
https://github.com/apache/spark/pull/14138#discussion_r70366868
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/CallMethodViaReflection.scala
---
@@ -0,0 +1,166 @@
+/*
+ * Lice
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14147
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62132/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14147
**[Test build #62132 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62132/consoleFull)**
for PR 14147 at commit
[`f86ea5a`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14147
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/14143
---
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 ena
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14138
**[Test build #62133 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62133/consoleFull)**
for PR 14138 at commit
[`ccfb9c4`](https://github.com/apache/spark/commit/c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14147
**[Test build #62132 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62132/consoleFull)**
for PR 14147 at commit
[`f86ea5a`](https://github.com/apache/spark/commit/f
GitHub user jkbradley opened a pull request:
https://github.com/apache/spark/pull/14147
[SPARK-14812][ML][MLLIB][PYTHON] Experimental, DeveloperApi annotation
audit for ML
## What changes were proposed in this pull request?
General decisions to follow, except where noted:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14138#discussion_r70366108
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/CallMethodViaReflection.scala
---
@@ -0,0 +1,166 @@
+/*
+ * Licens
Github user petermaxlee commented on a diff in the pull request:
https://github.com/apache/spark/pull/14138#discussion_r70366065
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CallMethodViaReflectionSuite.scala
---
@@ -0,0 +1,102 @@
+/*
+ *
Github user petermaxlee commented on a diff in the pull request:
https://github.com/apache/spark/pull/14138#discussion_r70366073
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/CallMethodViaReflection.scala
---
@@ -0,0 +1,166 @@
+/*
+ * Lice
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/14143
@zsxwing
I am merging this critical bug fix to master and 2.0. Feel free to leave
reviews and I will address them in follow up PRs.
---
If your project is set up for it, you can reply to t
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14138#discussion_r70365738
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CallMethodViaReflectionSuite.scala
---
@@ -0,0 +1,102 @@
+/*
+ * L
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14138#discussion_r70365507
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/CallMethodViaReflection.scala
---
@@ -0,0 +1,166 @@
+/*
+ * Licens
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14088
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62128/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14088
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/14088
**[Test build #62128 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62128/consoleFull)**
for PR 14088 at commit
[`d86ac9c`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14143
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62126/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14143
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/14143
**[Test build #62126 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62126/consoleFull)**
for PR 14143 at commit
[`a810dd4`](https://github.com/apache/spark/commit/
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13704#discussion_r70364606
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/SimplifyCastsSuite.scala
---
@@ -0,0 +1,78 @@
+/*
+ * Licensed to th
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13704#discussion_r70364539
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/PrimitiveArrayBenchmark.scala
---
@@ -0,0 +1,84 @@
+/*
+ * Licensed to
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14146
**[Test build #3184 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3184/consoleFull)**
for PR 14146 at commit
[`0df6a99`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14146
**[Test build #3183 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3183/consoleFull)**
for PR 14146 at commit
[`0df6a99`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14146
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62131/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14146
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/14100
---
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 ena
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13704#discussion_r70364232
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/SimplifyCastsSuite.scala
---
@@ -0,0 +1,78 @@
+/*
+ * Licensed to th
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/14146
This pr will fail until https://github.com/apache/spark/pull/14144 makes it
in.
---
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
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/14144#discussion_r70364188
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/test/DataFrameReaderWriterSuite.scala
---
@@ -449,6 +449,20 @@ class DataFrameReaderWriterSuite extends
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14097
---
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 ena
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/13704
@cloud-fan I added test cases. Could you review it again?
---
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
GitHub user rxin opened a pull request:
https://github.com/apache/spark/pull/14146
[SPARK-16489][SQL] checkEvaluation should fail if expression reuses
variable names
## What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/14100
LGTM. Merging to master and 2.0
---
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 wi
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/14097
LGTM. Merging to master and 2.0
---
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 wi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14145
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 rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/14144#discussion_r70363441
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/arithmetic.scala
---
@@ -498,34 +498,35 @@ case class Pmod(left: Expression, ri
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/14045
ping @liancheng @yhuai @rxin I think this is ready now. Can you review 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 pr
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/13957
---
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 ena
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14144
**[Test build #62129 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62129/consoleFull)**
for PR 14144 at commit
[`8e7b05f`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14045
**[Test build #62130 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62130/consoleFull)**
for PR 14045 at commit
[`42f53de`](https://github.com/apache/spark/commit/4
GitHub user ahmed-mahran opened a pull request:
https://github.com/apache/spark/pull/14145
[SPARK-16487] [STREAMING] Fix some batches might not get marked as fully
processed in JobGenerator
## What changes were proposed in this pull request?
In `JobGenerator`, the code read
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/13957
LGTM. Merging this to master and 2.0
---
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 a
Github user sameeragarwal commented on the issue:
https://github.com/apache/spark/pull/14144
cc @rxin
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/14088
**[Test build #62128 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62128/consoleFull)**
for PR 14088 at commit
[`d86ac9c`](https://github.com/apache/spark/commit/d
GitHub user sameeragarwal opened a pull request:
https://github.com/apache/spark/pull/14144
[SPARK-16488] Fix codegen variable namespace collision in pmod and
partitionBy
## What changes were proposed in this pull request?
This patch fixes a variable namespace collision bug
Github user sharkdtu commented on a diff in the pull request:
https://github.com/apache/spark/pull/14088#discussion_r70362297
--- Diff:
yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnClusterSuite.scala ---
@@ -274,6 +288,37 @@ private object YarnClusterDriverWithFailure exten
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13494#discussion_r70362160
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/OptimizeMetadataOnlyQuery.scala
---
@@ -0,0 +1,153 @@
+/*
+ * Licensed to the Apa
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13494#discussion_r70361858
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/OptimizeMetadataOnlyQuery.scala
---
@@ -0,0 +1,153 @@
+/*
+ * Licensed to the Apa
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14116
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62124/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14116
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/14116
**[Test build #62124 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62124/consoleFull)**
for PR 14116 at commit
[`550d69e`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14141
**[Test build #62127 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62127/consoleFull)**
for PR 14141 at commit
[`5122a1a`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11317
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/11317
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62122/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11317
**[Test build #62122 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62122/consoleFull)**
for PR 11317 at commit
[`f0051c4`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14139
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62123/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14139
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/14139
**[Test build #62123 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62123/consoleFull)**
for PR 14139 at commit
[`6708e9b`](https://github.com/apache/spark/commit/
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/14141
apparently dev/run-tests didn't actually finish, I'll fix the test failures
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14141
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62120/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14141
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/14141
**[Test build #62120 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62120/consoleFull)**
for PR 14141 at commit
[`7ac60de`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14112
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/14112
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62125/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14112
**[Test build #62125 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62125/consoleFull)**
for PR 14112 at commit
[`08e5b55`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14143
**[Test build #62126 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62126/consoleFull)**
for PR 14143 at commit
[`a810dd4`](https://github.com/apache/spark/commit/a
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/14080
There is a test failure
```
[error] Test
org.apache.spark.network.sasl.SaslIntegrationSuite.testAppIsolation failed:
java.lang.RuntimeException: java.lang.NullPointerException
[error]
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14080
**[Test build #3182 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3182/consoleFull)**
for PR 14080 at commit
[`258ed7a`](https://github.com/apache/spark/commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14035
Gentle ping @mengxr and @yanboliang
---
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
ena
GitHub user tdas opened a pull request:
https://github.com/apache/spark/pull/14143
[SPARK-16430][SQL][STREAMING] Fixed bug in the maxFilesPerTrigger in
FileStreamSource
## What changes were proposed in this pull request?
Incorrect list of files were being allocated to a bat
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/14116#discussion_r70353227
--- Diff: R/pkg/inst/tests/testthat/test_sparkSQL.R ---
@@ -502,20 +502,21 @@ test_that("jsonRDD() on a RDD with json string", {
})
test
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14112
**[Test build #62125 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62125/consoleFull)**
for PR 14112 at commit
[`08e5b55`](https://github.com/apache/spark/commit/0
Github user GayathriMurali commented on the issue:
https://github.com/apache/spark/pull/14112
+1 for separate loading logic. The recent commit includes separate code
paths depending on sparkVersion
---
If your project is set up for it, you can reply to this email and have your
reply
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14116#discussion_r70352851
--- Diff: R/pkg/inst/tests/testthat/test_sparkSQL.R ---
@@ -502,20 +502,21 @@ test_that("jsonRDD() on a RDD with json string", {
})
test_t
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14142
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 maver1ck opened a pull request:
https://github.com/apache/spark/pull/14142
[SPARK-16439] Fix number formatting in SQL UI
## What changes were proposed in this pull request?
Spark SQL UI display numbers greater than 1000 with u00A0 as grouping
separator.
Prob
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14116
**[Test build #62124 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62124/consoleFull)**
for PR 14116 at commit
[`550d69e`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14139
**[Test build #62123 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62123/consoleFull)**
for PR 14139 at commit
[`6708e9b`](https://github.com/apache/spark/commit/6
Github user hhbyyh commented on the issue:
https://github.com/apache/spark/pull/14112
I agree that it will make thing easier to separate the loading logic here
in LDA due to its extra complexity. Yet maybe we should not extend the pattern
to previous changes about loading compatibilit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14139
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/14139
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62121/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14139
**[Test build #62121 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62121/consoleFull)**
for PR 14139 at commit
[`308aef5`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11317
**[Test build #62122 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62122/consoleFull)**
for PR 11317 at commit
[`f0051c4`](https://github.com/apache/spark/commit/f
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14020
---
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 ena
101 - 200 of 660 matches
Mail list logo