Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15618
**[Test build #67491 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67491/consoleFull)**
for PR 15618 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15618
This PR simply rebases https://github.com/apache/spark/pull/12693 because
it seems almost there or already done but was not rebased and It has been stale.
I guess this should be
GitHub user HyukjinKwon opened a pull request:
https://github.com/apache/spark/pull/15618
[SPARK-14914][CORE] Fix Resource not closed after using, mostly for unit
tests
## What changes were proposed in this pull request?
Close `FileStreams`, `ZipFiles` etc to release the
Github user zhangxinyu1 closed the pull request at:
https://github.com/apache/spark/pull/15483
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15566
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15566
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67485/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15566
**[Test build #67485 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67485/consoleFull)**
for PR 15566 at commit
Github user zhangxinyu1 commented on the issue:
https://github.com/apache/spark/pull/15483
Sure. 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 wishes so,
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/15365#discussion_r84832132
--- Diff: R/pkg/R/mllib.R ---
@@ -647,6 +654,173 @@ setMethod("predict", signature(object =
"KMeansModel"),
predict_internal(object,
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/15365#discussion_r84832837
--- Diff: R/pkg/inst/tests/testthat/test_mllib.R ---
@@ -587,6 +587,59 @@ test_that("spark.isotonicRegression", {
unlink(modelPath)
})
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/15365#discussion_r84832641
--- Diff: R/pkg/R/mllib.R ---
@@ -647,6 +654,173 @@ setMethod("predict", signature(object =
"KMeansModel"),
predict_internal(object,
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/15365#discussion_r84833158
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/r/LogisticRegressionWrapper.scala ---
@@ -0,0 +1,163 @@
+/*
+ * Licensed to the Apache
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/15365#discussion_r84832848
--- Diff: R/pkg/inst/tests/testthat/test_mllib.R ---
@@ -587,6 +587,59 @@ test_that("spark.isotonicRegression", {
unlink(modelPath)
})
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/15365#discussion_r84832712
--- Diff: R/pkg/R/mllib.R ---
@@ -647,6 +654,173 @@ setMethod("predict", signature(object =
"KMeansModel"),
predict_internal(object,
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/15365#discussion_r84833023
--- Diff: R/pkg/R/mllib.R ---
@@ -647,6 +654,173 @@ setMethod("predict", signature(object =
"KMeansModel"),
predict_internal(object,
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/15365#discussion_r84832359
--- Diff: R/pkg/R/mllib.R ---
@@ -647,6 +654,195 @@ setMethod("predict", signature(object =
"KMeansModel"),
predict_internal(object,
Github user marmbrus commented on the issue:
https://github.com/apache/spark/pull/15483
I will try to take a look soon. Can we close this PR until we have an
updated implementation?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user chenghao-intel commented on the issue:
https://github.com/apache/spark/pull/15579
Oh, thank you @jerryshao , just noticed you gave inputs also. :)
---
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 zsxwing commented on the issue:
https://github.com/apache/spark/pull/15617
@kayousterhout no problem. yeah, the failure is unrelated.
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/15394
**[Test build #67488 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67488/consoleFull)**
for PR 15394 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15394
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67488/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15515
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15394
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15515
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67484/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15515
**[Test build #67484 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67484/consoleFull)**
for PR 15515 at commit
Github user davies commented on the issue:
https://github.com/apache/spark/pull/15484
@jiangxb1987 Could you say a little bit more about the "minor bug"? that
help us to decide this patch should be backported or not.
---
If your project is set up for it, you can reply to this email
Github user priyankagargnitk commented on the issue:
https://github.com/apache/spark/pull/15609
Hmm... I think that makes sense.. i am trying a couple of more things...
Thanks
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/15617
Thanks @zsxwing -- sorry about this! Looks like the test failure was
unrelated.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/15484
@davies Would you please have a look at this PR? Thank you!
---
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
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/15513#discussion_r84831284
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/xml/xpath.scala
---
@@ -95,8 +119,16 @@ case class XPathInt(xml:
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/15513#discussion_r84831067
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/xml/xpath.scala
---
@@ -95,8 +119,16 @@ case class XPathInt(xml:
Github user chenghao-intel commented on the issue:
https://github.com/apache/spark/pull/15579
@srowen
Besides `numactl`, some profiling tools like the `valgrind`, `strace`,
`vtune`, and also the system call hackings we probably needed before the
executor process launched.
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/15513#discussion_r84830885
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala
---
@@ -105,7 +113,15 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/1
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/1
**[Test build #67490 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67490/consoleFull)**
for PR 1 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/1
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67490/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/1
**[Test build #67490 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67490/consoleFull)**
for PR 1 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15612
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67483/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15612
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15024
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15024
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67489/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15024
**[Test build #67489 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67489/consoleFull)**
for PR 15024 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15612
**[Test build #67483 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67483/consoleFull)**
for PR 15612 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15516
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67487/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15516
**[Test build #67487 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67487/consoleFull)**
for PR 15516 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15516
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15513
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67482/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15513
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/12394
@jkbradley Sounds good to me!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #67482 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67482/consoleFull)**
for PR 15513 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15024
**[Test build #67489 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67489/consoleFull)**
for PR 15024 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15394
**[Test build #67488 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67488/consoleFull)**
for PR 15394 at commit
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/15394
Jenkins test 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
Github user keypointt commented on the issue:
https://github.com/apache/spark/pull/12195
I'll update the PR description if it looks good now
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/12195
**[Test build #67486 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67486/consoleFull)**
for PR 12195 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12195
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67486/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12195
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user zhangxinyu1 commented on the issue:
https://github.com/apache/spark/pull/15483
@marmbrus
I write a short deasign doc of KafkaSink in jira
https://issues.apache.org/jira/browse/SPARK-17935
Could you please take a look?
---
If your project is set up for it, you
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/15513#discussion_r84828020
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/arithmetic.scala
---
@@ -25,7 +25,11 @@ import
Github user sarutak commented on the issue:
https://github.com/apache/spark/pull/15611
Hey @hayashidac , you did't need to close this PR. You should have just
pushed your change...
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15516
**[Test build #67487 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67487/consoleFull)**
for PR 15516 at commit
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/15516
@felixcheung @yanboliang 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
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15024
> can you add a comment on CatalogStorageFormat that location.uri may
sometimes be resolved from table properties by the hive external catalog
Well, I'd like not to. This PR tries to
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12195
**[Test build #67486 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67486/consoleFull)**
for PR 12195 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15617
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15617
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67481/
Test FAILed.
---
Github user hayashidac commented on the issue:
https://github.com/apache/spark/pull/15611
Thank you for your comments. Iâll fix my program
---
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 hayashidac closed the pull request at:
https://github.com/apache/spark/pull/15611
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15617
**[Test build #67481 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67481/consoleFull)**
for PR 15617 at commit
Github user keypointt commented on the issue:
https://github.com/apache/spark/pull/12195
The main reason deleting those without duplication is because of the name,
but the actual code differs:
* StreamingLinearRegression
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/15579
@srowen, another use cases would be trace tools like `strace` which will
trace the system calls for process. One way of using `strace` is to add
`strace` before executing command.
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15513
(@gatorsmile BTW, Thank you for bearing with me)
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/15566
**[Test build #67485 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67485/consoleFull)**
for PR 15566 at commit
Github user keypointt commented on the issue:
https://github.com/apache/spark/pull/15353
sorry I was on vacation last 2 weeks. working on it now, please allow me
some time to get it done :)
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15610
@srowen, It seems we should fix `FastDateFormat.getInstance` too. It seems
that method also takes locale as the second argument.
```
.../sql/catalyst/json/JSONOptions.scala:
Github user koertkuipers commented on the issue:
https://github.com/apache/spark/pull/13526
@cloud-fan that makes sense to me, but its definitely not a quick win to
create that optimization.
let me think about it some more
---
If your project is set up for it, you can reply to
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15617
---
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 cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15566#discussion_r84824909
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -1363,21 +1377,34 @@ class DDLSuite extends QueryTest with
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/15617
LGTM. The compile is successful. I'm going to merge 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
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15513
Will review the remaining tomorrow.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15513#discussion_r84824488
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala
---
@@ -105,7 +113,15 @@ case class
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15513#discussion_r84824307
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala
---
@@ -76,7 +76,15 @@ case class
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15513#discussion_r84824076
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/conditionalExpressions.scala
---
@@ -24,7 +24,17 @@ import
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/15574
Other than that, this looks good to me.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15566#discussion_r84823617
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/InsertIntoHiveTableSuite.scala
---
@@ -370,17 +370,6 @@ class InsertIntoHiveTableSuite
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/15574
LGTM except for one minor comment.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15513#discussion_r84823563
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/arithmetic.scala
---
@@ -367,8 +428,18 @@ case class Remainder(left:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15024
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67479/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15024
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15024
**[Test build #67479 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67479/consoleFull)**
for PR 15024 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15596
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15596
**[Test build #67480 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67480/consoleFull)**
for PR 15596 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15515
**[Test build #67484 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67484/consoleFull)**
for PR 15515 at commit
Github user ericl commented on the issue:
https://github.com/apache/spark/pull/15515
Rebased
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15596
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67480/
Test PASSed.
---
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15566#discussion_r84823202
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala
---
@@ -366,26 +366,10 @@ case class
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/15612
@rxin Hope for your advice, 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
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15513#discussion_r84823116
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/arithmetic.scala
---
@@ -25,7 +25,11 @@ import
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15459
---
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 cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15459
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
1 - 100 of 602 matches
Mail list logo