Github user Yunni commented on a diff in the pull request:
https://github.com/apache/spark/pull/15148#discussion_r80411490
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/lsh/LSH.scala ---
@@ -0,0 +1,290 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/15224
Thanks. Merging to master and 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
Github user Yunni commented on a diff in the pull request:
https://github.com/apache/spark/pull/15148#discussion_r80411374
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/lsh/LSH.scala ---
@@ -0,0 +1,290 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/15238
cc @rxin
---
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 gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15233#discussion_r8044
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala
---
@@ -376,6 +376,10 @@ private[hive] class HiveClientImpl(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15238
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/15238
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65892/
Test PASSed.
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15107
@xwu0226 Can you please close this, please? @hvanhovell already added you
as the contributor in another PR, which has been merged. Thanks!
---
If your project is set up for it, you can reply to
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15238
**[Test build #65892 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65892/consoleFull)**
for PR 15238 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15216
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65895/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15216
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/15216
**[Test build #65895 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65895/consoleFull)**
for PR 15216 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14897
**[Test build #65896 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65896/consoleFull)**
for PR 14897 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12601
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65891/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12601
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/12601
**[Test build #65891 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65891/consoleFull)**
for PR 12601 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15168
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/15168
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65889/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15168
**[Test build #65889 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65889/consoleFull)**
for PR 15168 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15216
**[Test build #65895 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65895/consoleFull)**
for PR 15216 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15231
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/15231
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65894/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15231
**[Test build #65894 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65894/consoleFull)**
for PR 15231 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/14912
ping @cloud-fan @hvanhovell @srinathshankar again, please take look if you
have time. Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14897#discussion_r80407704
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala ---
@@ -60,20 +90,21 @@ case class CreateViewCommand(
child:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15231
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65893/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15231
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/15231
**[Test build #65893 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65893/consoleFull)**
for PR 15231 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15231
**[Test build #65894 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65894/consoleFull)**
for PR 15231 at commit
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/15189#discussion_r80406877
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/columnar/InMemoryColumnarQuerySuite.scala
---
@@ -232,4 +232,29 @@ class
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/15189#discussion_r80406261
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryRelation.scala
---
@@ -44,6 +44,70 @@ object InMemoryRelation {
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15231
(I just updated the PR description too)
---
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/15231
**[Test build #65893 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65893/consoleFull)**
for PR 15231 at commit
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/15090
To help us choose a better design, we need to first clarify the usage of
column stats.
A simple example may look like this (e.g. predicate: col < 5):
```java
filter.condition match {
Github user mortada commented on a diff in the pull request:
https://github.com/apache/spark/pull/15053#discussion_r80405104
--- Diff: python/pyspark/sql/functions.py ---
@@ -411,7 +415,7 @@ def monotonically_increasing_id():
The generated ID is guaranteed to be
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15238
**[Test build #65892 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65892/consoleFull)**
for PR 15238 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15231
**[Test build #65890 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65890/consoleFull)**
for PR 15231 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15231
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65890/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15231
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 viirya opened a pull request:
https://github.com/apache/spark/pull/15238
[SPARK-17653][SQL] Remove unnecessary distincts in multiple unions
## What changes were proposed in this pull request?
Currently for `Union [Distinct]`, a `Distinct` operator is necessary
Github user JustinPihony commented on the issue:
https://github.com/apache/spark/pull/12601
@srowen The doc changes have been reviewed, so this should be good to go
---
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/15231
**[Test build #65890 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65890/consoleFull)**
for PR 15231 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12601
**[Test build #65891 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65891/consoleFull)**
for PR 12601 at commit
Github user JustinPihony commented on a diff in the pull request:
https://github.com/apache/spark/pull/12601#discussion_r80404639
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/jdbc/JDBCWriteSuite.scala ---
@@ -208,4 +210,84 @@ class JDBCWriteSuite extends SharedSQLContext
Github user JustinPihony commented on a diff in the pull request:
https://github.com/apache/spark/pull/12601#discussion_r80404577
--- Diff: docs/sql-programming-guide.md ---
@@ -1096,13 +1096,17 @@ the Data Sources API. The following options are
supported:
{% highlight
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14897#discussion_r80403654
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -393,21 +459,25 @@ class SessionCatalog(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14897#discussion_r80403612
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -371,16 +431,24 @@ class SessionCatalog(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14897#discussion_r80403509
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -142,8 +149,12 @@ class SessionCatalog(
//
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14897#discussion_r80403416
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SharedState.scala ---
@@ -37,6 +37,20 @@ import
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14897#discussion_r80403386
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala ---
@@ -222,8 +265,8 @@ case class AlterViewAsCommand(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15168
**[Test build #65889 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65889/consoleFull)**
for PR 15168 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/15168
Definitely! Thank you, @gatorsmile . I added the logic to cover that.
---
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 cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14897#discussion_r80402604
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/GlobalTempViewManager.scala
---
@@ -0,0 +1,96 @@
+/*
+ * Licensed to
Github user chuanlei commented on a diff in the pull request:
https://github.com/apache/spark/pull/15195#discussion_r80401708
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/streaming/DataStreamWriter.scala
---
@@ -290,8 +284,8 @@ final class DataStreamWriter[T]
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15168
This PR needs to cover more negative cases. Below is an example:
```Scala
spark.range(10).select('id as 'a, 'id as 'b).createTempView("view1")
sql("DESC view1 PARTITION
Github user cjjnjust commented on a diff in the pull request:
https://github.com/apache/spark/pull/15172#discussion_r80399872
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/sasl/aes/SparkAesCipher.java
---
@@ -0,0 +1,270 @@
+/*
+ * Licensed to the
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/15195#discussion_r80396970
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/streaming/DataStreamWriter.scala
---
@@ -290,8 +284,8 @@ final class DataStreamWriter[T]
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/14897#discussion_r80396050
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SharedState.scala ---
@@ -37,6 +37,20 @@ import org.apache.spark.util.{MutableURLClassLoader,
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/14897#discussion_r80395764
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -453,7 +532,11 @@ class SessionCatalog(
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/14897#discussion_r80396017
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/CatalogImpl.scala ---
@@ -277,7 +275,7 @@ class CatalogImpl(sparkSession: SparkSession)
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/14897#discussion_r80395565
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -142,8 +149,12 @@ class SessionCatalog(
//
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/14897#discussion_r80395798
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala ---
@@ -222,8 +265,8 @@ case class AlterViewAsCommand(
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/14897#discussion_r80395752
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -393,21 +459,25 @@ class SessionCatalog(
*/
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/14897#discussion_r80395517
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/GlobalTempViewManager.scala
---
@@ -0,0 +1,96 @@
+/*
+ * Licensed to the
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/14897#discussion_r80395993
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/CatalogImpl.scala ---
@@ -277,7 +275,7 @@ class CatalogImpl(sparkSession: SparkSession)
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/14897#discussion_r80395571
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -47,6 +50,8 @@ object SessionCatalog {
*/
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/14897#discussion_r80395605
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -329,33 +343,77 @@ class SessionCatalog(
//
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/14897#discussion_r80395754
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -393,21 +459,25 @@ class SessionCatalog(
*/
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/14897#discussion_r80396066
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/GlobalTempViewSuite.scala
---
@@ -0,0 +1,107 @@
+/*
+ * Licensed to the Apache
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/14897#discussion_r80395510
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/GlobalTempViewManager.scala
---
@@ -0,0 +1,96 @@
+/*
+ * Licensed to the
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/14897#discussion_r80395895
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala ---
@@ -197,6 +201,45 @@ case class CreateViewCommand(
}
}
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/15226#discussion_r80395860
--- Diff:
core/src/main/scala/org/apache/spark/util/AsynchronousListenerBus.scala ---
@@ -117,6 +124,24 @@ private[spark] abstract class
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15123
---
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 hvanhovell commented on the issue:
https://github.com/apache/spark/pull/15123
LGTM - merging to master. Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15235
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65888/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15235
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/15235
**[Test build #65888 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65888/consoleFull)**
for PR 15235 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15231
Oh, BTW it seems `read.df` also seems not allowing this? I will try to test
and fix here together if so.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15231
@felixcheung , I usually don't like to answer by quote but let me do this
just to clarify.
> Hmm, should we hold till 12601 is merged then? Seems like we shouldn't
allow this unless
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/14897#discussion_r80393533
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -459,7 +459,8 @@ class Analyzer(
case u:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15237
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 erenavsarogullari opened a pull request:
https://github.com/apache/spark/pull/15237
[SPARK-17663] [CORE] SchedulableBuilder should handle invalid data access
via scheduler.alâ¦
## What changes were proposed in this pull request?
If `spark.scheduler.allocation.file`
Github user karlhigley commented on a diff in the pull request:
https://github.com/apache/spark/pull/15148#discussion_r80393070
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/lsh/LSH.scala ---
@@ -0,0 +1,290 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user karlhigley commented on a diff in the pull request:
https://github.com/apache/spark/pull/15148#discussion_r80392464
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/lsh/LSH.scala ---
@@ -0,0 +1,290 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user karlhigley commented on a diff in the pull request:
https://github.com/apache/spark/pull/15148#discussion_r80392692
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/lsh/LSH.scala ---
@@ -0,0 +1,290 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15235
**[Test build #65888 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65888/consoleFull)**
for PR 15235 at commit
Github user petermaxlee commented on the issue:
https://github.com/apache/spark/pull/15235
@brkyvz I think this also impacts the change you just did in
https://github.com/apache/spark/pull/15153. This change makes both code path
consistent.
---
If your project is set up for it,
Github user brkyvz commented on the issue:
https://github.com/apache/spark/pull/15153
@petermaxlee It is true that the parallel version can fail as well, the
same kind of race condition can bite people there
---
If your project is set up for it, you can reply to this email and have
Github user petermaxlee commented on the issue:
https://github.com/apache/spark/pull/15153
@brkyvz the change here only affects the serial version, and not the
parallel version, does it?
Wouldn't that be a problem?
---
If your project is set up for it, you can reply to
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15219
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65887/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15219
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/15219
**[Test build #65887 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65887/consoleFull)**
for PR 15219 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15232
Oh I meant 244.
---
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/15219
**[Test build #65887 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65887/consoleFull)**
for PR 15219 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/15097
@WeichenXu123 do you have the user code and sample data that when run with
SparkR will cause this issue? I think that will help us understand how this
happens better.
---
If your project is
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/15232
change LGTM to me.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/15232
you mean issue 224 of testthat on github? doesn't seem like it's related?
---
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 the issue:
https://github.com/apache/spark/pull/15231
Hmm, should we hold till 12601 is merged then? Seems like we shouldn't
allow this unless internal datasources are supporting this more broadly.
Also, before the path parameter type is
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/15216#discussion_r80388454
--- Diff: R/pkg/R/context.R ---
@@ -231,17 +231,21 @@ setCheckpointDir <- function(sc, dirName) {
#' filesystems), or an HTTP, HTTPS or FTP URI. To
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/15168
Thank you for review @gatorsmile .
Hi, @hvanhovell .
Could you review this again?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
1 - 100 of 197 matches
Mail list logo