Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14745
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/14728
**[Test build #64260 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64260/consoleFull)**
for PR 14728 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14745
**[Test build #64250 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64250/consoleFull)**
for PR 14745 at commit
Github user petermaxlee commented on a diff in the pull request:
https://github.com/apache/spark/pull/14728#discussion_r75795996
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/FileStreamSource.scala
---
@@ -17,21 +17,18 @@
package
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/8880
**[Test build #64259 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64259/consoleFull)**
for PR 8880 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/8880
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64259/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/8880
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 HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14743
Yup, I will organise my thought first with some researches and then open a
JIRA. Thanks.
---
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/8880
**[Test build #64259 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64259/consoleFull)**
for PR 8880 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14239
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/14239
**[Test build #64258 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64258/consoleFull)**
for PR 14239 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14750
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/14239
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64258/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14750
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64252/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14750
**[Test build #64252 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64252/consoleFull)**
for PR 14750 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14750
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/14750
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64251/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14750
**[Test build #64251 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64251/consoleFull)**
for PR 14750 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14239
**[Test build #64258 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64258/consoleFull)**
for PR 14239 at commit
Github user winningsix commented on a diff in the pull request:
https://github.com/apache/spark/pull/8880#discussion_r75794260
--- Diff:
core/src/main/scala/org/apache/spark/crypto/CryptoStreamUtils.scala ---
@@ -0,0 +1,104 @@
+/*
+ * Licensed to the Apache Software
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14618
do you mind update 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 feature
enabled
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14625
hmmm, can we split this PR into multiple PRs? We are not copying tests from
`HiveCompatibilitySuite` directly and it takes time to review each test and its
result.
---
If your project is set up
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14753#discussion_r75793140
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/interfaces.scala
---
@@ -389,3 +389,175 @@ abstract class
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14753#discussion_r75793025
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/interfaces.scala
---
@@ -389,3 +389,175 @@ abstract class
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14753#discussion_r75792744
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/interfaces.scala
---
@@ -389,3 +389,175 @@ abstract class
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14753#discussion_r75792506
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/interfaces.scala
---
@@ -389,3 +389,175 @@ abstract class
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14753#discussion_r75792480
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/interfaces.scala
---
@@ -389,3 +389,175 @@ abstract class
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14753#discussion_r75792451
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/interfaces.scala
---
@@ -389,3 +389,175 @@ abstract class
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/10896
okay, thanks. I'll fix in hours.
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14572
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/14572
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64244/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14572
**[Test build #64244 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64244/consoleFull)**
for PR 14572 at commit
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/14749
Hm why does this matter? Is it failing some tests?
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/14763
**[Test build #64257 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64257/consoleFull)**
for PR 14763 at commit
Github user VinceShieh commented on the issue:
https://github.com/apache/spark/pull/14640
@holdenk thanks for your comments. :) You are right. But as you can see,
this is a variant of kFold, so I think it's better to stay close to it,
otherwise, it would seems confusing, dont you
Github user JoshRosen commented on the issue:
https://github.com/apache/spark/pull/14763
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
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14757#discussion_r75791305
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/test/TestHive.scala ---
@@ -139,12 +139,9 @@ private[hive] class TestHiveSparkSession(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14447
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 cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14757#discussion_r75791279
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/test/TestHive.scala ---
@@ -139,12 +139,9 @@ private[hive] class TestHiveSparkSession(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14447
**[Test build #64247 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64247/consoleFull)**
for PR 14447 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14447
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64247/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14753
**[Test build #64256 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64256/consoleFull)**
for PR 14753 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14757#discussion_r75790997
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalogSuite.scala
---
@@ -40,6 +40,15 @@ abstract class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14764
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 cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14757#discussion_r75790947
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -41,13 +42,20 @@ import org.apache.spark.sql.types.{DataType,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14763
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/14763
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64246/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14763
**[Test build #64246 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64246/consoleFull)**
for PR 14763 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14764
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64249/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14764
**[Test build #64249 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64249/consoleFull)**
for PR 14764 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14764
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64248/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14764
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 VinceShieh commented on the issue:
https://github.com/apache/spark/pull/14640
if one understands the underlying ideas behind this method (labelKFold),
it's easy to take it as a class/category of data, though I do think it's not
that straightforward, even a bit confusing,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14764
**[Test build #64248 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64248/consoleFull)**
for PR 14764 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14753
**[Test build #64255 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64255/consoleFull)**
for PR 14753 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13988
**[Test build #64254 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64254/consoleFull)**
for PR 13988 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14752
**[Test build #64253 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64253/consoleFull)**
for PR 14752 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14752
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64245/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14752
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/14752
**[Test build #64245 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64245/consoleFull)**
for PR 14752 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14750
**[Test build #64252 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64252/consoleFull)**
for PR 14750 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14749
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/14750
**[Test build #64251 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64251/consoleFull)**
for PR 14750 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14749
thanks, merging to 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
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14745
@falaki and @felixcheung Do you mind if I ask another quick look 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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14745
**[Test build #64250 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64250/consoleFull)**
for PR 14745 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14764
**[Test build #64249 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64249/consoleFull)**
for PR 14764 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14764
**[Test build #64248 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64248/consoleFull)**
for PR 14764 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/14764
Follow up on #14758
---
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 felixcheung opened a pull request:
https://github.com/apache/spark/pull/14764
[SPARKR][MINOR] Update R DESCRIPTION file
## What changes were proposed in this pull request?
Update DESCRIPTION
## How was this patch tested?
Run install
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/14728#discussion_r75786696
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/FileStreamSource.scala
---
@@ -17,21 +17,18 @@
package
Github user junyangq commented on a diff in the pull request:
https://github.com/apache/spark/pull/14761#discussion_r75786649
--- Diff: R/pkg/R/utils.R ---
@@ -697,3 +697,20 @@ is_master_local <- function(master) {
is_sparkR_shell <- function() {
grepl(".*shell\\.R$",
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14447
**[Test build #64247 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64247/consoleFull)**
for PR 14447 at commit
Github user junyangq commented on a diff in the pull request:
https://github.com/apache/spark/pull/14761#discussion_r75786183
--- Diff: R/pkg/R/utils.R ---
@@ -697,3 +697,20 @@ is_master_local <- function(master) {
is_sparkR_shell <- function() {
grepl(".*shell\\.R$",
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14763
**[Test build #64246 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64246/consoleFull)**
for PR 14763 at commit
GitHub user JoshRosen opened a pull request:
https://github.com/apache/spark/pull/14763
[SPARK-17194] Use single quotes when generating SQL for string literals
When Spark emits SQL for a string literal, it should wrap the string in
single quotes, not double quotes. Databases which
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14079
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64242/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14079
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/14079
**[Test build #64242 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64242/consoleFull)**
for PR 14079 at commit
Github user clockfly commented on a diff in the pull request:
https://github.com/apache/spark/pull/14753#discussion_r75784293
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/interfaces.scala
---
@@ -389,3 +389,148 @@ abstract class
Github user liancheng commented on the issue:
https://github.com/apache/spark/pull/14749
@cloud-fan I noticed this issue because I saw `First` and `Last` are marked
as non-deterministic. Although we are not checking this property for now, it's
still good to set the right values.
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/14749#discussion_r75783595
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/collect.scala
---
@@ -54,6 +54,10 @@ abstract class Collect
Github user junyangq commented on a diff in the pull request:
https://github.com/apache/spark/pull/14761#discussion_r75783153
--- Diff: R/pkg/R/install.R ---
@@ -125,20 +127,24 @@ robust_download_tar <- function(mirrorUrl, version,
hadoopVersion, packageName,
message(msg)
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14447#discussion_r75783118
--- Diff: R/pkg/R/mllib.R ---
@@ -734,6 +830,28 @@ setMethod("write.ml", signature(object =
"KMeansModel", path = "character"),
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14759
---
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/14738
---
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 felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14761#discussion_r75782928
--- Diff: R/pkg/R/install.R ---
@@ -125,20 +127,24 @@ robust_download_tar <- function(mirrorUrl, version,
hadoopVersion, packageName,
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14757
is `getNewClient` equal to `getClient.newSession`?
---
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 shivaram commented on the issue:
https://github.com/apache/spark/pull/14759
Merging this to master, 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 feature
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14761#discussion_r75782708
--- Diff: R/pkg/R/utils.R ---
@@ -697,3 +697,20 @@ is_master_local <- function(master) {
is_sparkR_shell <- function() {
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14758
Hmm thats a good catch. Unfortunately it looks like CRAN only wants one
person / one email address to be in the Maintainer field. This is also
documented at
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14531#discussion_r75782639
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -75,18 +81,52 @@ case class CreateTableLikeCommand(
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/14738
Merged into 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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14707
---
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 felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14761#discussion_r75782412
--- Diff: R/pkg/R/sparkR.R ---
@@ -367,19 +367,21 @@ sparkR.session <- function(
overrideEnvs(sparkConfigMap, paramMap)
}
# do not
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14707
thanks, merging to 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
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14752#discussion_r75782141
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala
---
@@ -379,8 +379,9 @@ private[hive] class HiveClientImpl(
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/14759
LGTM
---
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 felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14759#discussion_r75781929
--- Diff: R/run-tests.sh ---
@@ -26,14 +26,35 @@ rm -f $LOGFILE
SPARK_TESTING=1 $FWDIR/../bin/spark-submit --driver-java-options
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/14759#discussion_r75781286
--- Diff: R/run-tests.sh ---
@@ -26,14 +26,35 @@ rm -f $LOGFILE
SPARK_TESTING=1 $FWDIR/../bin/spark-submit --driver-java-options
101 - 200 of 761 matches
Mail list logo
