Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/23072#discussion_r234432181
--- Diff: R/pkg/R/mllib_clustering.R ---
@@ -610,3 +616,57 @@ setMethod("write.ml", signature(object = "LDAModel",
path = "character"),
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/23072#discussion_r234432049
--- Diff: R/pkg/vignettes/sparkr-vignettes.Rmd ---
@@ -968,6 +970,17 @@ predicted <- predict(model, df)
head(predicted)
```
+
Github user uzadude commented on a diff in the pull request:
https://github.com/apache/spark/pull/23042#discussion_r234431689
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -138,6 +138,11 @@ object TypeCoercion {
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/23073#discussion_r234431864
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/ExecutorData.scala ---
@@ -27,12 +27,14 @@ import org.apache.spark.rpc.{RpcAddress,
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/23073
please put ^ comment into PR description (because comment is not included
in commit message once the PR is merged)
---
-
To
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23065
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98972/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23065
**[Test build #98972 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98972/testReport)**
for PR 23065 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23065
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23076
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23076
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23076
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/23072#discussion_r234432019
--- Diff: R/pkg/R/mllib_clustering.R ---
@@ -610,3 +616,57 @@ setMethod("write.ml", signature(object = "LDAModel",
path = "character"),
GitHub user DaveDeCaprio opened a pull request:
https://github.com/apache/spark/pull/23076
[SPARK-26103][SQL] Added maxDepth to limit the length of text plans
Nested query plans can get extremely large (hundreds of megabytes).
## What changes were proposed in this pull
Github user DaveDeCaprio commented on the issue:
https://github.com/apache/spark/pull/23076
This contribution is my original work and I license the work to the project
under the projectâs open source license.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23069
**[Test build #4432 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4432/testReport)**
for PR 23069 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23045
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23045
**[Test build #98955 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98955/testReport)**
for PR 23045 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22547
I was stuck with some personal business recently, I'll send a PR for batch
source after the weekend.
---
-
To unsubscribe,
GitHub user gengliangwang opened a pull request:
https://github.com/apache/spark/pull/23068
[SPARK-26098][WebUI] Show associated SQL query in Job page
## What changes were proposed in this pull request?
For jobs associated to SQL queries, it would be easier to understand
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23068
**[Test build #98958 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98958/testReport)**
for PR 23068 at commit
Github user gengliangwang commented on the issue:
https://github.com/apache/spark/pull/23068
@vanzin
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23068
**[Test build #98960 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98960/testReport)**
for PR 23068 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23064
Merged to master, branch-2.4 and branch-2.3.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23055
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23055
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23055
**[Test build #98962 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98962/testReport)**
for PR 23055 at commit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22779
@10110346 are you in a position to update this?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23049
**[Test build #98957 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98957/testReport)**
for PR 23049 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23049
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/23068#discussion_r234402077
--- Diff:
core/src/main/scala/org/apache/spark/status/AppStatusListener.scala ---
@@ -70,6 +70,8 @@ private[spark] class AppStatusListener(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23068
**[Test build #98959 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98959/testReport)**
for PR 23068 at commit
Github user gengliangwang commented on the issue:
https://github.com/apache/spark/pull/23068
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/23064
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user shahidki31 commented on a diff in the pull request:
https://github.com/apache/spark/pull/23038#discussion_r234400684
--- Diff:
core/src/main/scala/org/apache/spark/status/AppStatusListener.scala ---
@@ -565,7 +567,11 @@ private[spark] class AppStatusListener(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/23054#discussion_r234401319
--- Diff: docs/sql-migration-guide-upgrade.md ---
@@ -17,6 +17,8 @@ displayTitle: Spark SQL Upgrading Guide
- The `ADD JAR` command
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23054
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23054
**[Test build #98961 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98961/testReport)**
for PR 23054 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23054
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/23042#discussion_r234403917
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -138,6 +138,11 @@ object TypeCoercion {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23066
**[Test build #4428 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4428/testReport)**
for PR 23066 at commit
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/23062#discussion_r234404845
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/ScalaReflection.scala
---
@@ -788,12 +788,37 @@ object ScalaReflection extends
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/23062#discussion_r234404786
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -498,6 +504,7 @@ case class NewInstance(
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/23062#discussion_r234404797
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -453,7 +453,7 @@ case class NewInstance(
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/23062#discussion_r234404871
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/ScalaReflection.scala
---
@@ -788,12 +788,37 @@ object ScalaReflection extends
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/23062#discussion_r234404858
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/ScalaReflection.scala
---
@@ -788,12 +788,37 @@ object ScalaReflection extends
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/23062#discussion_r234404888
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/ScalaReflection.scala
---
@@ -788,12 +788,37 @@ object ScalaReflection extends
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23049
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98954/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23066
**[Test build #98949 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98949/testReport)**
for PR 23066 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23055
**[Test build #98952 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98952/testReport)**
for PR 23055 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23045
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23066
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23066
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98949/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23055
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23038
**[Test build #98953 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98953/testReport)**
for PR 23038 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23049
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23038
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23038
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98953/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23055
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98952/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23049
**[Test build #98954 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98954/testReport)**
for PR 23049 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23038
**[Test build #98956 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98956/testReport)**
for PR 23038 at commit
Github user gengliangwang commented on the issue:
https://github.com/apache/spark/pull/23049
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23049
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/23042#discussion_r234401696
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -138,6 +138,11 @@ object TypeCoercion {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23068
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23068
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/23068#discussion_r234402149
--- Diff: core/src/main/scala/org/apache/spark/status/AppStatusStore.scala
---
@@ -56,6 +56,11 @@ private[spark] class AppStatusStore(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23068
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23068
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98958/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23068
**[Test build #98958 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98958/testReport)**
for PR 23068 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23068
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23068
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23055
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23069
**[Test build #98964 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98964/testReport)**
for PR 23069 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23069
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98964/
Test FAILed.
---
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/23059
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23070
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23069
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23070
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23045
**[Test build #98955 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98955/testReport)**
for PR 23045 at commit
Github user 10110346 commented on the issue:
https://github.com/apache/spark/pull/22779
@srowen Thanks, I am sorry, I am on holiday, l will update it next week ,I
am reply ing on my phone.
---
-
To unsubscribe,
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23070
add to whitelist
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23070
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23049
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98957/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23049
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23054
**[Test build #98961 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98961/testReport)**
for PR 23054 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23038
**[Test build #98968 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98968/testReport)**
for PR 23038 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23068
**[Test build #98959 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98959/testReport)**
for PR 23068 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23068
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23068
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98959/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23068
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23068
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98960/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23057
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23057
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23057
**[Test build #98969 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98969/testReport)**
for PR 23057 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23055
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98962/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23055
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23055
**[Test build #98962 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98962/testReport)**
for PR 23055 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23070
**[Test build #98963 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98963/testReport)**
for PR 23070 at commit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/23016
Merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
GitHub user MaxGekk opened a pull request:
https://github.com/apache/spark/pull/23070
[SPARK-26099][SQL] Verification of the corrupt column in from_csv/from_json
## What changes were proposed in this pull request?
The corrupt column specified via JSON/CSV option
1 - 100 of 199 matches
Mail list logo
