Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22274
@felixcheung From system preference -
Time zone : Pacific Daylight Time
closest city :Cupertino, CA, United States
---
--
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/22274
maybe also your laptop's system time zone? could you also check that?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/22274#discussion_r214530330
--- Diff: R/pkg/tests/fulltests/test_sparkSQL.R ---
@@ -3633,7 +3633,8 @@ test_that("catalog APIs, currentDatabase,
setCurrentDatabase, listDatabases",
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/22295#discussion_r214530177
--- Diff: python/pyspark/sql/session.py ---
@@ -252,6 +252,16 @@ def newSession(self):
"""
return self.__class__(self._sc, self
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/22298#discussion_r214530079
--- Diff: examples/src/main/python/worker_memory_check.py ---
@@ -0,0 +1,47 @@
+#
--- End diff --
shouldn't this be in python tests (and
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22270
@ueshin Opened (https://github.com/apache/spark/pull/22314) and
(https://github.com/apache/spark/pull/22315). Thank you.
---
---
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/7#discussion_r214529571
--- Diff: python/pyspark/sql/functions.py ---
@@ -1669,20 +1669,36 @@ def repeat(col, n):
return Column(sc._jvm.functions.repeat(_to_java_column
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22314
@kiszk Thanks. Can you please help look at
[22315](https://github.com/apache/spark/pull/22315) as well.
---
-
To unsubscribe
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22315
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22315
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/2764/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22315
**[Test build #95586 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95586/testReport)**
for PR 22315 at commit
[`84b135c`](https://github.com/apache/spark/commit/84
GitHub user dilipbiswal opened a pull request:
https://github.com/apache/spark/pull/22315
[SPARK-25308][SQL] ArrayContains function may return a error in the code
generation phase.
## What changes were proposed in this pull request?
Invoking ArrayContains function with non nulla
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22313
Thank you for review, @kiszk .
First, I don't want to hold the memory up after query completion. If we do,
it will be a regression. So, I wanted `time` first.
Second, It's dif
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22179
Thanks, @wangyum !
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revi
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/22179
@dongjoon-hyun I'm trying to add test cases.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional co
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/22313
General question: Why do we use `time` instead of `entry size` to control
cache? I am neutral on this decision. I would like to hear the reason of this
decision.
---
---
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/22314
Good catch, LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22313#discussion_r214528778
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/orc/OrcFilters.scala ---
@@ -55,19 +59,52 @@ import org.apache.spark.sql.types._
* k
Github user xuanyuanking commented on a diff in the pull request:
https://github.com/apache/spark/pull/22313#discussion_r214528435
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/orc/OrcFilters.scala ---
@@ -55,19 +59,52 @@ import org.apache.spark.sql.types._
* kn
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22314
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/2763/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22314
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22314
**[Test build #95585 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95585/testReport)**
for PR 22314 at commit
[`7dec581`](https://github.com/apache/spark/commit/7d
GitHub user dilipbiswal opened a pull request:
https://github.com/apache/spark/pull/22314
[SPARK-25307] ArraySort function may return an error in the code generation
phase
## What changes were proposed in this pull request?
Sorting array of booleans (not nullable) returns a comp
Github user sadhen commented on the issue:
https://github.com/apache/spark/pull/22308
@cloud-fan @srowen
This PR will make our jenkins build green.
https://amplab.cs.berkeley.edu/jenkins/view/Spark%20QA%20Test/job/spark-master-test-maven-hadoop-2.7-ubuntu-scala-2.12/
Github user sadhen commented on the issue:
https://github.com/apache/spark/pull/22308
see https://github.com/apache/spark/pull/12924 and
https://github.com/apache/spark/pull/11744
---
-
To unsubscribe, e-mail: revie
Github user sadhen commented on the issue:
https://github.com/apache/spark/pull/22310
@srowen It is about the sbt convention.
see my demo project: https://github.com/sadhen/spark-25298
---
-
To unsubscribe,
Github user sadhen commented on the issue:
https://github.com/apache/spark/pull/22308
Remove the deprecated 2.10 jar.
And build a 2.12 jar.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22262
@seancxmao . Could you explain why we need this PR? Apache Spark 2.3.1
already shows exceptions like the following for the case (for both ORC and
Parquet), doesn't it?
```scala
sc
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22313
Could you review this PR, @gatorsmile and @cloud-fan ?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
F
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22270
@ueshin Thank you.. will do.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-ma
Github user NiharS commented on the issue:
https://github.com/apache/spark/pull/22192
Tried on a different machine and those tests pass, here's hoping
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.or
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22313
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22313
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95583/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22313
**[Test build #95583 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95583/testReport)**
for PR 22313 at commit
[`ac06b0c`](https://github.com/apache/spark/commit/a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22192
**[Test build #95584 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95584/testReport)**
for PR 22192 at commit
[`5a2852f`](https://github.com/apache/spark/commit/5a
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/7#discussion_r214526544
--- Diff:
common/unsafe/src/test/java/org/apache/spark/unsafe/types/UTF8StringSuite.java
---
@@ -394,12 +394,14 @@ public void substringSQL() {
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/22270
@dilipbiswal Let's fix the two functions in 2.4. Could you open a pr to fix
the two functions? Thanks!
---
-
To unsubscribe, e-ma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22312
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95582/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22312
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22312
**[Test build #95582 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95582/testReport)**
for PR 22312 at commit
[`5772404`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22312
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95581/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22312
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22312
**[Test build #95581 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95581/testReport)**
for PR 22312 at commit
[`ad3a11d`](https://github.com/apache/spark/commit/a
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22236
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/21721
I spent more hours to take a look at how SQL UI can update the metrics
information before task ends, and now I guess I may understand what was the
concern from @cloud-fan here.
This is
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22236
Merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22179
Ping, @wangyum . Could you add the test cases for SPARK-20389, SPARK-23131,
and SPARK-25176?
---
-
To unsubscribe, e-mail:
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22179#discussion_r214524736
--- Diff: pom.xml ---
@@ -1770,6 +1770,10 @@
org.apache.hive
hive-storage-api
+
+
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22313
**[Test build #95583 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95583/testReport)**
for PR 22313 at commit
[`ac06b0c`](https://github.com/apache/spark/commit/ac
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22313
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/2762/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22313
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
GitHub user dongjoon-hyun opened a pull request:
https://github.com/apache/spark/pull/22313
[SPARK-25306][SQL] Use cache to speed up `createFilter`
## What changes were proposed in this pull request?
In ORC data source, `createFilter` function has exponential time complexity
Github user LucaCanali commented on the issue:
https://github.com/apache/spark/pull/22218
Thanks @srowen
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22310
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22310
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95579/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22310
**[Test build #95579 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95579/testReport)**
for PR 22310 at commit
[`7c0b9b9`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/7
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95578/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22312
**[Test build #95582 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95582/testReport)**
for PR 22312 at commit
[`5772404`](https://github.com/apache/spark/commit/57
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/7
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22312
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/2761/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/7
**[Test build #95578 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95578/testReport)**
for PR 7 at commit
[`d17d2df`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22312
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22218#discussion_r214522065
--- Diff:
core/src/main/scala/org/apache/spark/executor/ExecutorSource.scala ---
@@ -73,6 +76,28 @@ class ExecutorSource(threadPool: ThreadPoolExecutor,
exe
Github user LucaCanali commented on the issue:
https://github.com/apache/spark/pull/22218
I have implemented the changes as from the latest comments by @maropu and
@srowen
---
-
To unsubscribe, e-mail: reviews-unsu
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22312
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/2760/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22312
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user koertkuipers commented on the issue:
https://github.com/apache/spark/pull/21273
@HyukjinKwon see https://github.com/apache/spark/pull/22312
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.o
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22312
**[Test build #95581 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95581/testReport)**
for PR 22312 at commit
[`ad3a11d`](https://github.com/apache/spark/commit/ad
GitHub user koertkuipers opened a pull request:
https://github.com/apache/spark/pull/22312
[SPARK-17916][SQL] Fix new behavior when quote is set and fix old behavior
when quote is unset
## What changes were proposed in this pull request?
1) Set nullValue to quoted empty str
Github user LucaCanali commented on a diff in the pull request:
https://github.com/apache/spark/pull/22218#discussion_r214521537
--- Diff:
core/src/main/scala/org/apache/spark/executor/ExecutorSource.scala ---
@@ -73,6 +75,29 @@ class ExecutorSource(threadPool: ThreadPoolExecutor,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22112
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95577/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22112
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22112
**[Test build #95577 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95577/testReport)**
for PR 22112 at commit
[`63b3eb6`](https://github.com/apache/spark/commit/6
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22310
Why the move of the source code in the 2.11 tree? looks like it's not in
the right package dir now.
---
-
To unsubscribe, e-mail:
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22308
Oh what's the change here in the .jar file? just copied the jar file for
2.11/2.10?
---
-
To unsubscribe, e-mail: reviews-unsubs
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22311
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95580/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22311
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22311
**[Test build #95580 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95580/testReport)**
for PR 22311 at commit
[`f94fdf7`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22308
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22308
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95576/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22308
**[Test build #95576 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95576/testReport)**
for PR 22308 at commit
[`221fcac`](https://github.com/apache/spark/commit/2
Github user gengliangwang commented on the issue:
https://github.com/apache/spark/pull/22311
@cloud-fan @gatorsmile @wangyum
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/22311#discussion_r214518048
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -649,17 +648,20 @@ object CollapseProject extend
Github user gengliangwang commented on the issue:
https://github.com/apache/spark/pull/22311
Thanks @wangyum for finding the issue. Before this PR, he propose
https://github.com/apache/spark/pull/22124 and
https://github.com/apache/spark/pull/22287 .
But there is no description f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22311
**[Test build #95580 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95580/testReport)**
for PR 22311 at commit
[`f94fdf7`](https://github.com/apache/spark/commit/f9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22311
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/2759/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22311
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
GitHub user gengliangwang opened a pull request:
https://github.com/apache/spark/pull/22311
[SPARK-25305][SQL] Respect attribute name in CollapseProject and
ColumnPruning
## What changes were proposed in this pull request?
Currently in optimizer rule `CollapseProject`, the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22310
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22310
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user sadhen commented on the issue:
https://github.com/apache/spark/pull/22310
@srowen This is a better solution for #22280
No hard-coded 2.11 or 2.12 any more.
---
-
To unsubscribe, e-mail: reviews
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22310
**[Test build #95579 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95579/testReport)**
for PR 22310 at commit
[`7c0b9b9`](https://github.com/apache/spark/commit/7c
GitHub user sadhen opened a pull request:
https://github.com/apache/spark/pull/22310
[Spark-25298][Build] Improve build definition for Scala 2.12
## What changes were proposed in this pull request?
Improve build for Scala 2.12.
## How was this patch tested?
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22309
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22309
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22309
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user mt40 opened a pull request:
https://github.com/apache/spark/pull/22309
[SPARK-20384][core] Support value class in schema of Dataset
## What changes were proposed in this pull request?
This PR adds support for [Scala value class][1] in schema of Datasets (as
both top
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/7
**[Test build #95578 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95578/testReport)**
for PR 7 at commit
[`d17d2df`](https://github.com/apache/spark/commit/d1
Github user phegstrom commented on a diff in the pull request:
https://github.com/apache/spark/pull/7#discussion_r214516705
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/regexpExpressions.scala
---
@@ -229,36 +229,74 @@ case class RLike(left:
Github user phegstrom commented on a diff in the pull request:
https://github.com/apache/spark/pull/7#discussion_r214516644
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/regexpExpressions.scala
---
@@ -229,36 +229,74 @@ case class RLike(left:
1 - 100 of 137 matches
Mail list logo