Github user sameeragarwal commented on a diff in the pull request:
https://github.com/apache/spark/pull/15319#discussion_r83791645
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/QueryPlan.scala
---
@@ -74,14 +74,26 @@ abstract class QueryPlan[PlanType <:
Github user wangmiao1981 commented on the issue:
https://github.com/apache/spark/pull/15421
Update R on another linux:
R version 3.3.1 (2016-06-21) -- "Bug in Your Hair"
Copyright (C) 2016 The R Foundation for Statistical Computing
Platform: x86_64-pc-linux-gnu (64-bit)
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15266#discussion_r83788778
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/rules.scala
---
@@ -252,6 +251,8 @@ case class PreprocessTableInsertion(conf:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15528
**[Test build #67114 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67114/consoleFull)**
for PR 15528 at commit
[`5994f07`](https://github.com/apache/spark/commit/5
GitHub user cloud-fan opened a pull request:
https://github.com/apache/spark/pull/15528
[SPARK-17899][SQL][follow-up] debug mode should work for corrupted table
## What changes were proposed in this pull request?
Debug mode should work for corrupted table, so that we can rea
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15528
@yhuai @gatorsmile @ericl
---
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 wangmiao1981 commented on the issue:
https://github.com/apache/spark/pull/15421
You are right. There is no change for the two versions.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/15219
@davies could you please let us know if you need additional information for
starting your review?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15527
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/15527
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67113/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15527
**[Test build #67113 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67113/consoleFull)**
for PR 15527 at commit
[`6c8d459`](https://github.com/apache/spark/commit/
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/15421
Sorry my question wasn't clear - Is there a source change that we can spot
that might have caused this behavior ? I don't see this line changing recently
looking at history of
https://github.com/w
Github user leoromanovsky commented on a diff in the pull request:
https://github.com/apache/spark/pull/15526#discussion_r83785993
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/feature/SQLTransformerSuite.scala ---
@@ -43,6 +43,7 @@ class SQLTransformerSuite
assert(re
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/9
We should change the `initMode` doc to indicate that the param is ignored
when `initialModel` is set.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/15495
@gatorsmile @yhuai Many 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 does not have this feature
enabl
Github user wangmiao1981 commented on the issue:
https://github.com/apache/spark/pull/15421
As discussed previously, R 3.3.1 works. For 3.3.0, `NA` is serialized but
it is not serialized as `String`.
---
If your project is set up for it, you can reply to this email and have your
re
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/15421
did this change in a recent R version ? I'm not sure why `NA` is not being
serialized ? That `if` statement should only affect the value assigned to
`type` right ?
---
If your project is set up
Github user wangmiao1981 commented on the issue:
https://github.com/apache/spark/pull/15421
@shivaram I think we can use that test case. Somehow, I missed the debug
message of [3] and [4], but it should not be quite related. The reason should
be my `serialize` function, as shown above
Github user wangmiao1981 commented on the issue:
https://github.com/apache/spark/pull/15421
I think the reason is because of the code below:
`> serialize
function (object, connection, ascii = FALSE, xdr = TRUE, version = NULL,
refhook = NULL)
{
if (!is
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15527
**[Test build #67113 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67113/consoleFull)**
for PR 15527 at commit
[`6c8d459`](https://github.com/apache/spark/commit/6
GitHub user koeninger opened a pull request:
https://github.com/apache/spark/pull/15527
[SPARK-17813][SQL][KAFKA] Maximum data per trigger
## What changes were proposed in this pull request?
maxOffsetsPerTrigger option for rate limiting, proportionally based on
volume of di
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15520
**[Test build #67112 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67112/consoleFull)**
for PR 15520 at commit
[`6aa9e2f`](https://github.com/apache/spark/commit/6
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/15520
cc @andrewor14
---
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/15417
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67108/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15417
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/15417
**[Test build #67108 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67108/consoleFull)**
for PR 15417 at commit
[`59cf500`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15525
**[Test build #67111 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67111/consoleFull)**
for PR 15525 at commit
[`f318dff`](https://github.com/apache/spark/commit/f
Github user loneknightpy commented on the issue:
https://github.com/apache/spark/pull/15285
@tdas Addressed your comments, please take a look.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/15525
Jenkins, 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 wis
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15285
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67106/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15285
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/15285
**[Test build #67106 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67106/consoleFull)**
for PR 15285 at commit
[`82d4575`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15481
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/15481
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67105/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15481
**[Test build #67105 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67105/consoleFull)**
for PR 15481 at commit
[`2997ccb`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15526
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 drewrobb opened a pull request:
https://github.com/apache/spark/pull/15526
[SPARK-17986] [ML] SQLTransformer should remove temporary tables
## What changes were proposed in this pull request?
A call to the method `SQLTransformer.transform` previously would create
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/15500#discussion_r83781558
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/basicPhysicalOperators.scala
---
@@ -77,9 +77,40 @@ case class ProjectExec(projectList:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15525
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/15525
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67107/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15519
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/15519
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67104/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15525
**[Test build #67107 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67107/consoleFull)**
for PR 15525 at commit
[`f318dff`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15519
**[Test build #67104 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67104/consoleFull)**
for PR 15519 at commit
[`3229095`](https://github.com/apache/spark/commit/
Github user viirya closed the pull request at:
https://github.com/apache/spark/pull/15500
---
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 jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/15417#discussion_r83781091
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/ResolveNaturalJoinSuite.scala
---
@@ -31,39 +31,46 @@ class ResolveNaturalJ
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/15417#discussion_r83781262
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/BooleanSimplificationSuite.scala
---
@@ -91,51 +99,51 @@ class BooleanSimp
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/15500#discussion_r83781352
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/basicPhysicalOperators.scala
---
@@ -77,9 +77,40 @@ case class ProjectExec(projectList:
Seq
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/15417#discussion_r83781173
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/AggregateOptimizeSuite.scala
---
@@ -58,9 +58,9 @@ class AggregateOptimize
Github user yhuai commented on the issue:
https://github.com/apache/spark/pull/15518
Seems this fails the scala style check.
---
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
enab
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15518
Oops - just realized the tests for the latest commit failed. I will revert
the patch.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. I
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15266
Any other comment? @cloud-fan 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 does not have this feature
en
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15417
@jiangxb1987 Can you leave a comment on the PR changes to explain why you
made these changes? You know, reviewing these changes is not easy. Thanks!
---
If your project is set up for it, you can
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15459
@yhuai Any further comment about it? 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 does not have this fea
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15512
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67102/
Test PASSed.
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15523
cc @cloud-fan @davies @sameeragarwal
---
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
en
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15512
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/15512
**[Test build #67102 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67102/consoleFull)**
for PR 15512 at commit
[`7d73691`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15505
**[Test build #67110 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67110/consoleFull)**
for PR 15505 at commit
[`ca9da40`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15505
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67110/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15505
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/15505
**[Test build #67110 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67110/consoleFull)**
for PR 15505 at commit
[`ca9da40`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14847
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/14847
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67103/
Test PASSed.
---
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15518
---
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/14847
**[Test build #67103 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67103/consoleFull)**
for PR 14847 at commit
[`beb1b45`](https://github.com/apache/spark/commit/
Github user koeninger commented on the issue:
https://github.com/apache/spark/pull/15407
@rxin @tdas right now, items to be committed can be added to the queue, but
they will never actually be removed from the queue. poll() removes, iterator()
does not. I updated the description of
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15518
LGTM - merging in 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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15505
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/15505
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67109/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15505
**[Test build #67109 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67109/consoleFull)**
for PR 15505 at commit
[`d956ff5`](https://github.com/apache/spark/commit/
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/15471
@falaki looks like the SparkR MLlib unit tests are timing out on Jenkins.
Do they pass on your machine ?
---
If your project is set up for it, you can reply to this email and have your
reply appea
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15407
@koeninger can you put more information into the description of the pull
request? At the very least we should talk about the current implementation
causes memory leaks.
---
If your project is set up
Github user witgo commented on the issue:
https://github.com/apache/spark/pull/15505
@wzhfy
Ok, the code has been modified
---
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
en
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15505
**[Test build #67109 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67109/consoleFull)**
for PR 15505 at commit
[`d956ff5`](https://github.com/apache/spark/commit/d
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15423#discussion_r8378
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -651,14 +651,28 @@ case class ShowTablePropertiesCommand(table
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15495
Merging to master! 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 does not have this feature
enabled and w
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15495
---
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/15500#discussion_r83777528
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/basicPhysicalOperators.scala
---
@@ -77,9 +77,40 @@ case class ProjectExec(projectList:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15517#discussion_r83776817
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/QueryExecution.scala ---
@@ -61,17 +61,16 @@ class QueryExecution(val sparkSession: SparkS
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15519
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67101/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15519
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/15519
**[Test build #67101 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67101/consoleFull)**
for PR 15519 at commit
[`4ce3093`](https://github.com/apache/spark/commit/
Github user witgo commented on the issue:
https://github.com/apache/spark/pull/15512
I also think that the time saved is all the registration which can be
skipped, but did not verify.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15417
**[Test build #67108 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67108/consoleFull)**
for PR 15417 at commit
[`59cf500`](https://github.com/apache/spark/commit/5
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/15319
This PR is ready for review, would anyone look at 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 hav
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/15417
@gatorsmile This PR is ready for review.
---
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 featur
Github user witgo commented on a diff in the pull request:
https://github.com/apache/spark/pull/15512#discussion_r83774753
--- Diff: core/src/main/scala/org/apache/spark/scheduler/TaskResult.scala
---
@@ -77,14 +78,12 @@ private[spark] class DirectTaskResult[T](
*
*
Github user witgo commented on a diff in the pull request:
https://github.com/apache/spark/pull/15512#discussion_r83774768
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskResultGetter.scala ---
@@ -84,6 +90,7 @@ private[spark] class TaskResultGetter(sparkEnv: SparkEnv,
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/14136
@rxin What do we need to update to make this PR accepted? Please give some
advice, many thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15525
**[Test build #67107 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67107/consoleFull)**
for PR 15525 at commit
[`f318dff`](https://github.com/apache/spark/commit/f
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/15414
@jkbradley @yanboliang Could you please have a review of this? This PR
unify usage of labelCol casting and fixs a bug described in
[https://issues.apache.org/jira/browse/SPARK-17797]
---
If y
GitHub user ueshin opened a pull request:
https://github.com/apache/spark/pull/15525
[SPARK-17985][CORE] Bump commons-lang3 version to 3.5.
## What changes were proposed in this pull request?
`SerializationUtils.clone()` of commons-lang3 (<3.5) has a bug that breaks
thread
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/15522#discussion_r83773809
--- Diff: docs/sql-programming-guide.md ---
@@ -422,8 +422,8 @@ In the simplest form, the default data source
(`parquet` unless otherwise config
You
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/15505
There are many unnecessary changes, can you recover them to minimize diff?
That'll be easier for others to review. :)
---
If your project is set up for it, you can reply to this email and have your
r
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15481
**[Test build #67105 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67105/consoleFull)**
for PR 15481 at commit
[`2997ccb`](https://github.com/apache/spark/commit/2
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15285
**[Test build #67106 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67106/consoleFull)**
for PR 15285 at commit
[`82d4575`](https://github.com/apache/spark/commit/8
Github user scwf commented on the issue:
https://github.com/apache/spark/pull/15481
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, or
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/15407
Can you explain what is the memory that can currently leak with the
iterator?
---
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 p
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15519
**[Test build #67104 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67104/consoleFull)**
for PR 15519 at commit
[`3229095`](https://github.com/apache/spark/commit/3
1 - 100 of 549 matches
Mail list logo