Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/16670#discussion_r98267056
--- Diff: R/pkg/R/utils.R ---
@@ -756,12 +756,12 @@ varargsToJProperties <- function(...) {
props
}
-launchScript <- function(script,
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/16720
@shivaram
---
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 felixcheung opened a pull request:
https://github.com/apache/spark/pull/16720
[SPARK-19387][SPARKR] Tests do not run with SparkR source package in CRAN
check
## What changes were proposed in this pull request?
- this is cause by changes in SPARK-18444,
Github user kevinyu98 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16712#discussion_r98079046
--- Diff:
sql/core/src/test/resources/sql-tests/results/subquery/scalar-subquery/scalar-subquery-select.sql.out
---
@@ -0,0 +1,198 @@
+--
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/16720
Hmm - another fix could be that in the test cases whenever we create
`spark.session` we always pass in `master=local` ?
---
If your project is set up for it, you can reply to this email and have
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16720
**[Test build #72082 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72082/testReport)**
for PR 16720 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16694
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/16694
**[Test build #72080 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72080/testReport)**
for PR 16694 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16694
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72080/
Test PASSed.
---
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/16706
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,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16706
**[Test build #72083 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72083/testReport)**
for PR 16706 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16686
**[Test build #72081 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72081/testReport)**
for PR 16686 at commit
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/16670#discussion_r98264755
--- Diff: R/pkg/inst/tests/testthat/test_Windows.R ---
@@ -20,7 +20,7 @@ test_that("sparkJars tag in SparkContext", {
if (.Platform$OS.type !=
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16686
**[Test build #72081 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72081/testReport)**
for PR 16686 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16694
**[Test build #72080 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72080/testReport)**
for PR 16694 at commit
Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/14725#discussion_r98257954
--- Diff: python/pyspark/ml/wrapper.py ---
@@ -16,6 +16,10 @@
#
from abc import ABCMeta, abstractmethod
+import sys
--- End diff
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/16625
@sarutak want to look at this since you looked at the somewhat mildly
related PR?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16719
**[Test build #72076 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72076/testReport)**
for PR 16719 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/16717
Merging to 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 wishes so, or
Github user vanzin closed the pull request at:
https://github.com/apache/spark/pull/16717
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16719
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72076/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16719
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 zsxwing commented on the issue:
https://github.com/apache/spark/pull/16706
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,
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/16709
merged to master, branch-2.1, branch-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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16709
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16686
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72081/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16720
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 dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/16719#discussion_r98278557
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ExpressionSetSuite.scala
---
@@ -75,10 +107,14 @@ class
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/16719#discussion_r98280054
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Canonicalize.scala
---
@@ -78,14 +78,18 @@ object Canonicalize
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16497
**[Test build #72079 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72079/testReport)**
for PR 16497 at commit
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/16650#discussion_r98283223
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/BlacklistTrackerSuite.scala ---
@@ -17,7 +17,8 @@
package org.apache.spark.scheduler
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/16650#discussion_r98281415
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/BlacklistTracker.scala ---
@@ -187,6 +199,17 @@ private[scheduler] class BlacklistTracker (
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/16650#discussion_r98281426
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/BlacklistTracker.scala ---
@@ -187,6 +199,17 @@ private[scheduler] class BlacklistTracker (
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16686
**[Test build #72085 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72085/testReport)**
for PR 16686 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16686
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 dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/16719
Hi, @lw-lin .
Thank you for pining me. I'll 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
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/16720
yes, that is described in the PR description:
"
fix is to add check to the beginning of each test and vignettes; the same
would also work by changing `sparkR.session()` to
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/16720
I think better would be the approach being taken in PR 16330 - has a first
run test that prepare and run these kind of thing
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16670
**[Test build #72084 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72084/testReport)**
for PR 16670 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16720
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72082/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16720
**[Test build #72082 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72082/testReport)**
for PR 16720 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16497
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72079/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16497
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 dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/16719
The original logic was designed to be safe for changing the caller
bottom-up code,
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/16650#discussion_r98283147
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/BlacklistTrackerSuite.scala ---
@@ -456,4 +459,61 @@ class BlacklistTrackerSuite extends
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/16650#discussion_r98281375
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/BlacklistTracker.scala ---
@@ -173,6 +174,17 @@ private[scheduler] class BlacklistTracker (
Github user squito commented on the issue:
https://github.com/apache/spark/pull/16650
I think this is basically ready, perhaps one more test case. @tgravescs
any other comments?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16670
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72084/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16670
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/16670
**[Test build #72084 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72084/testReport)**
for PR 16670 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16686
**[Test build #72086 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72086/testReport)**
for PR 16686 at commit
GitHub user felixcheung opened a pull request:
https://github.com/apache/spark/pull/16721
[SPARKR][DOCS] update R API doc for subset/extract
## What changes were proposed in this pull request?
With extract `[[` or replace `[[<-`, the parameter `i` is a column index,
that
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16721
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72089/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16686
**[Test build #72085 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72085/testReport)**
for PR 16686 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16686
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/16686
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72085/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16706
**[Test build #72083 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72083/testReport)**
for PR 16706 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16706
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/16706
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72083/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16721
**[Test build #72090 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72090/testReport)**
for PR 16721 at commit
Github user actuaryzhang commented on the issue:
https://github.com/apache/spark/pull/16699
@zhengruifeng Thanks for the suggestions. Added casting and
instrumentation.
@imatiach-msft Thanks for the clarification! It is probably worth another
PR to clean up all tests in GLM.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16724
**[Test build #72096 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72096/testReport)**
for PR 16724 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16700
---
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 imatiach-msft commented on the issue:
https://github.com/apache/spark/pull/16699
@actuaryzhang thank you for updating the PR - I agree with your first two
points. With regards to the method of RDD[Instance] also working with
RDD[OffsetInstance], I think it should work,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16686
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72086/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16686
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/16686
**[Test build #72086 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72086/testReport)**
for PR 16686 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16706
---
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 julienledem commented on the issue:
https://github.com/apache/spark/pull/16281
FYI: Parquet 1.8.2 vote thread passed:
https://mail-archives.apache.org/mod_mbox/parquet-dev/201701.mbox/%3CCAO4re1mHLT%2BLYn8s1RTEDZK8-9WSVugY8-HQqAN%2BtU%3DBOi1L9w%40mail.gmail.com%3E
---
If
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/16694
LGTM, thank you!
Merging with 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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16722
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/16722
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72093/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16722
**[Test build #72093 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72093/testReport)**
for PR 16722 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16700#discussion_r98317696
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -839,6 +839,26 @@ private[spark] class
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/15768
Btw, @yanboliang and @Yunni did you sync? I'm fine with the takeover, but
don't want to stomp on toes. Both can be listed as authors when this gets
merged. Should we close this issue with the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14725
**[Test build #72092 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72092/testReport)**
for PR 14725 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16722
**[Test build #72091 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72091/testReport)**
for PR 16722 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16722
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72091/
Test FAILed.
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16636
: ) Done. Found a solution to infer the schema of Hive Serde tables. Let me
clean the code now.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16650
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/16043
**[Test build #72088 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72088/testReport)**
for PR 16043 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16650
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72087/
Test PASSed.
---
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/16690#discussion_r98297209
--- Diff: core/src/main/scala/org/apache/spark/rpc/RpcEndpointRef.scala ---
@@ -63,8 +65,48 @@ private[spark] abstract class RpcEndpointRef(conf:
SparkConf)
Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/16699#discussion_r98299104
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/regression/GeneralizedLinearRegression.scala
---
@@ -336,14 +361,15 @@ class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16721
**[Test build #72090 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72090/testReport)**
for PR 16721 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16721
**[Test build #72089 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72089/testReport)**
for PR 16721 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16722
**[Test build #72093 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72093/testReport)**
for PR 16722 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16700#discussion_r98317671
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -839,6 +839,26 @@ private[spark] class
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16700#discussion_r98317719
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -899,6 +919,21 @@ private[spark] class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16650
**[Test build #72087 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72087/testReport)**
for PR 16650 at commit
Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/16699#discussion_r98300999
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/regression/GeneralizedLinearRegressionSuite.scala
---
@@ -743,6 +743,84 @@ class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16721
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72090/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16721
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/16722
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 jkbradley opened a pull request:
https://github.com/apache/spark/pull/16723
[SPARK-19389][ML][PYTHON][DOC] Minor doc fixes for ML Python Params and
LinearSVC
## What changes were proposed in this pull request?
* Removed Since tags in Python Params since they
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16723
**[Test build #72094 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72094/testReport)**
for PR 16723 at commit
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/16723
@wangmiao1981 Would you mind checking this? It has small fixes I noticed
when reviewing your PR for Python LinearSVC.
---
If your project is set up for it, you can reply to this email and have
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16700#discussion_r98317878
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -899,6 +919,21 @@ private[spark] class
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16700#discussion_r98317808
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -899,6 +919,21 @@ private[spark] class
Github user tcondie commented on the issue:
https://github.com/apache/spark/pull/16686
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
1 - 100 of 265 matches
Mail list logo
