Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16841
**[Test build #72984 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72984/testReport)**
for PR 16841 at commit
Github user budde commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r101461535
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -161,23 +161,49 @@ private[hive] class
Github user budde commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r101461357
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveSchemaInferenceSuite.scala
---
@@ -0,0 +1,162 @@
+/*
+ * Licensed to the Apache
Github user budde commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r101461155
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveSchemaInferenceSuite.scala
---
@@ -0,0 +1,162 @@
+/*
+ * Licensed to the Apache
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/16620
@kayousterhout @squito @markhamstra
Thanks for all of your work for this patch. Really appreciate your help : )
---
If your project is set up for it, you can reply to this email and have
Github user budde commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r101460842
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -161,23 +161,49 @@ private[hive] class
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r101460558
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1802,4 +1806,118 @@ class JsonSuite extends
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16776
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/16776
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72983/
Test PASSed.
---
Github user budde commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r101460565
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -296,6 +296,17 @@ object SQLConf {
.longConf
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16776
**[Test build #72983 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72983/testReport)**
for PR 16776 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r101460391
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1764,4 +1769,117 @@ class JsonSuite extends
Github user pjfanning commented on a diff in the pull request:
https://github.com/apache/spark/pull/16895#discussion_r101460138
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1364,10 +1364,11 @@ class JsonSuite extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16947
**[Test build #72988 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72988/testReport)**
for PR 16947 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16953
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
GitHub user stanzhai opened a pull request:
https://github.com/apache/spark/pull/16953
[SPARK-19622][WebUI]Fix a http error in a paged table when using a `Go`
button to search.
## What changes were proposed in this pull request?
The search function of paged table is not
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r101457828
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveSchemaInferenceSuite.scala
---
@@ -0,0 +1,162 @@
+/*
+ * Licensed to the Apache
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r101457755
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveSchemaInferenceSuite.scala
---
@@ -0,0 +1,162 @@
+/*
+ * Licensed to the Apache
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r101457348
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveSchemaInferenceSuite.scala
---
@@ -0,0 +1,162 @@
+/*
+ * Licensed to the Apache
Github user pjfanning commented on a diff in the pull request:
https://github.com/apache/spark/pull/16895#discussion_r101456955
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1364,10 +1364,11 @@ class JsonSuite extends
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r101456952
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -161,23 +161,49 @@ private[hive] class
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r101456890
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -161,23 +161,49 @@ private[hive] class
Github user actuaryzhang commented on the issue:
https://github.com/apache/spark/pull/16729
@felixcheung Thanks for the discussions. Will work on this in two weeks.
---
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 viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r101456600
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -161,23 +161,49 @@ private[hive] class
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r101456277
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -186,8 +212,7 @@ private[hive] class
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r101454895
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -296,6 +296,17 @@ object SQLConf {
.longConf
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16895#discussion_r101454773
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1364,10 +1364,11 @@ class JsonSuite extends
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16948
---
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 the issue:
https://github.com/apache/spark/pull/15125
I think @mallman is saying he would merge changes to @dding3 branch
---
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 felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/16951#discussion_r101454231
--- Diff: R/pkg/R/stats.R ---
@@ -149,15 +149,18 @@ setMethod("freqItems", signature(x =
"SparkDataFrame", cols = "character"),
#' This method
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/16951#discussion_r101454291
--- Diff: R/pkg/R/stats.R ---
@@ -149,15 +149,18 @@ setMethod("freqItems", signature(x =
"SparkDataFrame", cols = "character"),
#' This method
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/16948
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 felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/16951#discussion_r101454138
--- Diff: R/pkg/R/stats.R ---
@@ -149,15 +149,18 @@ setMethod("freqItems", signature(x =
"SparkDataFrame", cols = "character"),
#' This method
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/16386
LGTM if the test can pass. It will be good if you can also address
https://github.com/apache/spark/pull/16386#discussion_r100679183
---
If your project is set up for it, you can reply to this
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16909#discussion_r101454158
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ExternalAppendOnlyUnsafeRowArray.scala
---
@@ -0,0 +1,218 @@
+/*
+ * Licensed to
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16951
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72985/
Test PASSed.
---
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/16951#discussion_r101453993
--- Diff: R/pkg/R/stats.R ---
@@ -149,15 +149,18 @@ setMethod("freqItems", signature(x =
"SparkDataFrame", cols = "character"),
#' This method
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16952
**[Test build #72987 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72987/testReport)**
for PR 16952 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r101453945
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/streaming/DataStreamReader.scala
---
@@ -151,6 +153,8 @@ final class DataStreamReader
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16951
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/16951
**[Test build #72985 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72985/testReport)**
for PR 16951 at commit
GitHub user carsonwang opened a pull request:
https://github.com/apache/spark/pull/16952
[SPARK-19620][SQL]Fix incorrect exchange coordinator id in the physical plan
## What changes were proposed in this pull request?
When adaptive execution is enabled, an exchange coordinator
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r101453441
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameReader.scala ---
@@ -332,20 +336,21 @@ class DataFrameReader private[sql](sparkSession:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r101453350
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameReader.scala ---
@@ -261,14 +261,18 @@ class DataFrameReader private[sql](sparkSession:
Github user uncleGen commented on the issue:
https://github.com/apache/spark/pull/16949
cc @srowen
---
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
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/16948
@cloud-fan : this is as per our discussion in
https://github.com/apache/spark/pull/16931#discussion_r101170361
---
If your project is set up for it, you can reply to this email and have your
Github user kevinyu98 commented on the issue:
https://github.com/apache/spark/pull/16915
@gatorsmile thanks a lot.
---
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/16386#discussion_r101451711
--- Diff: python/pyspark/sql/streaming.py ---
@@ -442,6 +444,8 @@ def json(self, path, schema=None,
primitivesAsString=None, prefersDecimal=None,
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r101451677
--- Diff: python/pyspark/sql/readwriter.py ---
@@ -159,18 +159,21 @@ def json(self, path, schema=None,
primitivesAsString=None, prefersDecimal=None,
Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/16620
LGTM! Thanks for finding this subtle bug and all of the hard work to fix it
@jinxing64. I'll wait until tomorrow to merge this to give Mark and Imran a
chance for any last comments.
---
If
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16949
**[Test build #72986 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72986/testReport)**
for PR 16949 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16818#discussion_r101451202
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/expressions/WindowSpec.scala ---
@@ -180,16 +180,20 @@ class WindowSpec private[sql](
Github user uncleGen commented on the issue:
https://github.com/apache/spark/pull/16949
jenkins crushed. 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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16951
**[Test build #72985 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72985/testReport)**
for PR 16951 at commit
GitHub user yanboliang opened a pull request:
https://github.com/apache/spark/pull/16951
[SPARK-19619][SPARKR] SparkR approxQuantile supports input multiple columns
## What changes were proposed in this pull request?
SparkR ```approxQuantile``` supports input multiple columns.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16949
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72979/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16949
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/16949
**[Test build #72979 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72979/testReport)**
for PR 16949 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16944
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72978/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16944
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 sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/16722#discussion_r101449037
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tree/impl/BaggedPoint.scala ---
@@ -60,12 +68,14 @@ private[spark] object BaggedPoint {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16944
**[Test build #72978 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72978/testReport)**
for PR 16944 at commit
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/16722#discussion_r101448976
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tree/impl/BaggedPoint.scala ---
@@ -82,16 +92,16 @@ private[spark] object BaggedPoint {
val
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/16722#discussion_r101448658
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/DecisionTreeClassifierSuite.scala
---
@@ -351,6 +370,36 @@ class
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/16722#discussion_r101448626
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tree/impl/DecisionTreeMetadata.scala
---
@@ -115,7 +122,10 @@ private[spark] object DecisionTreeMetadata
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16844
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/16844
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72977/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16844
**[Test build #72977 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72977/testReport)**
for PR 16844 at commit
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14412#discussion_r101447842
--- Diff:
core/src/main/scala/org/apache/spark/storage/BlockManagerMasterEndpoint.scala
---
@@ -65,6 +66,8 @@ class BlockManagerMasterEndpoint(
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16499#discussion_r101448019
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala
---
@@ -813,7 +813,14 @@ private[spark] class BlockManager(
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14412#discussion_r101447797
--- Diff:
core/src/main/scala/org/apache/spark/storage/BlockManagerMasterEndpoint.scala
---
@@ -195,17 +198,39 @@ class BlockManagerMasterEndpoint(
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14412#discussion_r101447703
--- Diff:
core/src/main/scala/org/apache/spark/storage/BlockManagerMasterEndpoint.scala
---
@@ -195,17 +198,39 @@ class BlockManagerMasterEndpoint(
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14412#discussion_r101446619
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala
---
@@ -1131,14 +1131,43 @@ private[spark] class BlockManager(
}
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/16950
merged to branch-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
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14412#discussion_r101447299
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala
---
@@ -1131,14 +1131,43 @@ private[spark] class BlockManager(
}
Github user felixcheung closed the pull request at:
https://github.com/apache/spark/pull/16950
---
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 JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14412#discussion_r101447394
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala
---
@@ -1131,14 +1131,43 @@ private[spark] class BlockManager(
}
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14412#discussion_r101446632
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala
---
@@ -1131,14 +1131,43 @@ private[spark] class BlockManager(
}
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14412#discussion_r101446759
--- Diff:
core/src/main/scala/org/apache/spark/storage/BlockManagerMasterEndpoint.scala
---
@@ -195,17 +198,39 @@ class BlockManagerMasterEndpoint(
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14412#discussion_r101446961
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala
---
@@ -1131,14 +1131,43 @@ private[spark] class BlockManager(
}
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14412#discussion_r101447547
--- Diff:
core/src/main/scala/org/apache/spark/storage/BlockManagerMasterEndpoint.scala
---
@@ -195,17 +198,39 @@ class BlockManagerMasterEndpoint(
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16915
---
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 JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14412#discussion_r101447632
--- Diff:
core/src/main/scala/org/apache/spark/storage/BlockManagerMasterEndpoint.scala
---
@@ -195,17 +198,39 @@ class BlockManagerMasterEndpoint(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16950
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72982/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16950
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 gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16915
LGTM. 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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/16950
**[Test build #72982 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72982/testReport)**
for PR 16950 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16841
**[Test build #72984 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72984/testReport)**
for PR 16841 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16499#discussion_r101447383
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala
---
@@ -1018,7 +1025,9 @@ private[spark] class BlockManager(
try {
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16841
ok to test
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16776
**[Test build #72983 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72983/testReport)**
for PR 16776 at commit
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/15468
@sameeragarwal would it be possible to review this since I resolved a
conflict?
---
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 felixcheung commented on the issue:
https://github.com/apache/spark/pull/16945
could you add some tests (I don't think we need one for each) that the
weight col is set properly?
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16620
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72974/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16620
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/16620
**[Test build #72974 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72974/testReport)**
for PR 16620 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16934
---
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 zsxwing commented on the issue:
https://github.com/apache/spark/pull/16934
Thanks! Merging to master and 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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16950
**[Test build #72982 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72982/testReport)**
for PR 16950 at commit
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/16512
At least from the R code perspective this is source code compatible with
existing 2.1, as adding an optional parameter at the end should not break any
existing code. Also I am not sure I would
1 - 100 of 587 matches
Mail list logo
