Github user sameeragarwal commented on a diff in the pull request:
https://github.com/apache/spark/pull/17733#discussion_r113377134
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -277,43 +279,73 @@ class Dataset[T] private[sql](
val sb = n
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/17733#discussion_r113376654
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -277,43 +279,73 @@ class Dataset[T] private[sql](
val sb = new Stri
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17733#discussion_r113375813
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -277,43 +279,73 @@ class Dataset[T] private[sql](
val sb = new
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17640#discussion_r113372131
--- Diff: R/pkg/R/serialize.R ---
@@ -83,6 +83,7 @@ writeObject <- function(con, object, writeType = TRUE) {
Date = writeDate(con, object),
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17640#discussion_r113370732
--- Diff: R/pkg/inst/tests/testthat/test_sparkSQL.R ---
@@ -3043,6 +3043,23 @@ test_that("catalog APIs, currentDatabase,
setCurrentDatabase, listDatabas
Github user gatorsmile closed the pull request at:
https://github.com/apache/spark/pull/17760
---
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 gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17649#discussion_r113373050
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/InMemoryCatalog.scala
---
@@ -295,7 +295,9 @@ class InMemoryCatalog(
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17649
@wzhfy In HiveClientImpl.scala, we follow Hive and consider the case
sensitivity of property key `comment`.
@sujith71955 Could you resolve the following comment?
https://github.com/
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/17733#discussion_r113371912
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -277,43 +279,73 @@ class Dataset[T] private[sql](
val sb = new Stri
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17596
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/17596
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76169/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17596
**[Test build #76169 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76169/testReport)**
for PR 17596 at commit
[`4df95f2`](https://github.com/apache/spark/commit/4
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17733#discussion_r113370773
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -277,43 +279,73 @@ class Dataset[T] private[sql](
val sb = new
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17758
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76167/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17758
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/17758
**[Test build #76167 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76167/testReport)**
for PR 17758 at commit
[`05a7a61`](https://github.com/apache/spark/commit/0
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17768
@joshrosen, could you take a look and see if it makes sense?
---
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 the issue:
https://github.com/apache/spark/pull/17768
This actually also happens when the directory exists but the user does not
have the permission.
---
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/17768
**[Test build #76172 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76172/testReport)**
for PR 17768 at commit
[`bf21e3b`](https://github.com/apache/spark/commit/bf
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17768
**[Test build #76171 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76171/testReport)**
for PR 17768 at commit
[`b9ce248`](https://github.com/apache/spark/commit/b9
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/17737
@holdenk do you have bandwidth to review this or ok with me pushing this to
master?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17130#discussion_r113366111
--- Diff: mllib/src/main/scala/org/apache/spark/ml/fpm/FPGrowth.scala ---
@@ -268,12 +269,8 @@ class FPGrowthModel private[ml] (
val predictUDF
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17728
---
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 HyukjinKwon opened a pull request:
https://github.com/apache/spark/pull/17768
[SPARK-20465][CORE] Throws a proper exception when any temp directory could
not be got/created (rather than ArrayIndexOutOfBoundsException)
## What changes were proposed in this pull request?
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/17728
merged 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 wishes so,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17191
**[Test build #76170 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76170/testReport)**
for PR 17191 at commit
[`7b32f46`](https://github.com/apache/spark/commit/7b
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/17191#discussion_r113365310
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -136,6 +136,7 @@ class Analyzer(
ResolveGroupi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17757
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76168/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17757
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/17757
**[Test build #76168 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76168/testReport)**
for PR 17757 at commit
[`a87d5c0`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17596
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76157/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17596
**[Test build #76157 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76157/testReport)**
for PR 17596 at commit
[`7f41155`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17596
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 felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17757#discussion_r113362917
--- Diff: R/pkg/inst/tests/testthat/test_mllib_classification.R ---
@@ -284,22 +284,11 @@ test_that("spark.mlp", {
c("1.0", "1.0", "1
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17757#discussion_r113362748
--- Diff: R/pkg/inst/tests/testthat/test_mllib_classification.R ---
@@ -284,22 +284,11 @@ test_that("spark.mlp", {
c("1.0", "1.0", "1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17733
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/17733
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76166/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17733
**[Test build #76166 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76166/testReport)**
for PR 17733 at commit
[`f696d35`](https://github.com/apache/spark/commit/f
Github user wangmiao1981 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17640#discussion_r113362108
--- Diff: R/pkg/inst/tests/testthat/test_Serde.R ---
@@ -28,6 +28,10 @@ test_that("SerDe of primitive types", {
expect_equal(x, 1)
expect_
Github user facaiy commented on the issue:
https://github.com/apache/spark/pull/17503
@srowen I am not sure whether I understand your question clearly.
RandomForest uses LearningNode to construct tree model when training, and
convert them to Leaf or InternalNode at last. Hence, all
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/17757#discussion_r113361559
--- Diff: R/pkg/inst/tests/testthat/test_mllib_classification.R ---
@@ -284,22 +284,11 @@ test_that("spark.mlp", {
c("1.0", "1.0", "1.
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17191#discussion_r113360559
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -136,6 +136,7 @@ class Analyzer(
ResolveGro
Github user facaiy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17503#discussion_r113360409
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/tree/configuration/Strategy.scala
---
@@ -61,6 +61,8 @@ import org.apache.spark.mllib.tree.impurity.{E
Github user ebernhardson commented on a diff in the pull request:
https://github.com/apache/spark/pull/16618#discussion_r113360277
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/evaluation/RankingMetrics.scala ---
@@ -0,0 +1,202 @@
+/*
+ * Licensed to the Apache Softwar
Github user ebernhardson commented on a diff in the pull request:
https://github.com/apache/spark/pull/16618#discussion_r113358325
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/evaluation/RankingEvaluator.scala ---
@@ -0,0 +1,138 @@
+/*
+ * Licensed to the Apache Softw
Github user ebernhardson commented on a diff in the pull request:
https://github.com/apache/spark/pull/16618#discussion_r113355473
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/evaluation/RankingMetrics.scala ---
@@ -0,0 +1,202 @@
+/*
+ * Licensed to the Apache Softwar
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17596
**[Test build #76169 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76169/testReport)**
for PR 17596 at commit
[`4df95f2`](https://github.com/apache/spark/commit/4d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17757
**[Test build #76168 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76168/testReport)**
for PR 17757 at commit
[`a87d5c0`](https://github.com/apache/spark/commit/a8
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17693
---
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 the issue:
https://github.com/apache/spark/pull/17693
thanks, merging to master/2.2
---
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 facaiy commented on the issue:
https://github.com/apache/spark/pull/17556
fix failed case, please retest 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 have this feature
enabled a
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17760
thanks, merging to 2.1
---
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 wishe
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17758
**[Test build #76167 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76167/testReport)**
for PR 17758 at commit
[`05a7a61`](https://github.com/apache/spark/commit/05
Github user wangmiao1981 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17640#discussion_r113358460
--- Diff: R/pkg/inst/tests/testthat/test_Serde.R ---
@@ -28,6 +28,10 @@ test_that("SerDe of primitive types", {
expect_equal(x, 1)
expect_
Github user wangmiao1981 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17640#discussion_r113358355
--- Diff: R/pkg/inst/tests/testthat/test_sparkSQL.R ---
@@ -3043,6 +3043,23 @@ test_that("catalog APIs, currentDatabase,
setCurrentDatabase, listDataba
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17758
**[Test build #76165 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76165/testReport)**
for PR 17758 at commit
[`de11a5b`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17758
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76165/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17758
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/17765
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/17765
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76163/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17765
**[Test build #76163 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76163/testReport)**
for PR 17765 at commit
[`bd13a01`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17765
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/17765
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76162/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17765
**[Test build #76162 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76162/testReport)**
for PR 17765 at commit
[`609d50e`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17765
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/17765
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76160/
Test PASSed.
---
Github user jinxing64 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17744#discussion_r113356306
--- Diff:
common/network-shuffle/src/main/java/org/apache/spark/network/shuffle/ExternalShuffleBlockHandler.java
---
@@ -93,14 +92,25 @@ protected void h
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17765
**[Test build #76160 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76160/testReport)**
for PR 17765 at commit
[`bd13a01`](https://github.com/apache/spark/commit/b
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/17649
The changes look good to me if we don't care about the case sensitivity
issue.
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17766
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76154/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17725
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/17766
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/17725
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76156/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17766
**[Test build #76154 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76154/testReport)**
for PR 17766 at commit
[`d16be2b`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17725
**[Test build #76156 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76156/testReport)**
for PR 17725 at commit
[`80e40ba`](https://github.com/apache/spark/commit/8
Github user hhbyyh commented on the issue:
https://github.com/apache/spark/pull/17767
Preparing a PR like this takes a lot of efforts. Please try to follow the
guidelines in http://spark.apache.org/contributing.html. (create a jira and
rename the title).
Like you said, I dou
Github user unsleepy22 commented on the issue:
https://github.com/apache/spark/pull/17738
Could someone 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 this feature
enabled and w
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17733#discussion_r113351508
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -663,8 +695,54 @@ class Dataset[T] private[sql](
* @group action
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17733
**[Test build #76166 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76166/testReport)**
for PR 17733 at commit
[`f696d35`](https://github.com/apache/spark/commit/f6
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/17191#discussion_r113351326
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -136,6 +136,7 @@ class Analyzer(
ResolveGroupi
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17760
cc @cloud-fan @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
enabled an
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17693
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76153/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17693
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/17693
**[Test build #76153 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76153/testReport)**
for PR 17693 at commit
[`91bb487`](https://github.com/apache/spark/commit/9
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17737#discussion_r113350427
--- Diff: python/pyspark/sql/column.py ---
@@ -251,15 +285,16 @@ def __iter__(self):
# string methods
_rlike_doc = """
-R
Github user zero323 commented on the issue:
https://github.com/apache/spark/pull/17077
ð
---
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 feat
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/16781#discussion_r113346208
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/ParquetHiveCompatibilitySuite.scala
---
@@ -397,13 +392,38 @@ class ParquetHiveCompatibilitySuit
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/16781#discussion_r113345272
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/ParquetHiveCompatibilitySuite.scala
---
@@ -17,14 +17,25 @@
package org.apache.spark.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17758
**[Test build #76165 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76165/testReport)**
for PR 17758 at commit
[`de11a5b`](https://github.com/apache/spark/commit/de
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17767
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 danielyli opened a pull request:
https://github.com/apache/spark/pull/17767
Als refactor
## What changes were proposed in this pull request?
This is a non-feature-changing refactoring of the ALS code (specifically,
the `org.apache.spark.ml.recommendation` packag
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17605
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76164/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17605
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/17605
**[Test build #76164 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76164/testReport)**
for PR 17605 at commit
[`9d75094`](https://github.com/apache/spark/commit/9
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17077
I think we should because branch-2.2 is cut out.
---
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 thi
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/17733#discussion_r113347799
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -663,8 +695,54 @@ class Dataset[T] private[sql](
* @group action
*
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/17733#discussion_r113347676
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -663,8 +695,54 @@ class Dataset[T] private[sql](
* @group action
*
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/17733#discussion_r113347666
--- Diff: R/pkg/R/DataFrame.R ---
@@ -194,6 +194,8 @@ setMethod("isLocal",
#' 20 characters will be truncated. However, if set
greater t
Github user zero323 commented on the issue:
https://github.com/apache/spark/pull/17077
@holdenk, @HyukjinKwon Do we retarget this to 2.3?
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17728
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
1 - 100 of 329 matches
Mail list logo