Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/18317#discussion_r138502374
--- Diff: core/src/main/java/org/apache/spark/io/ReadAheadInputStream.java
---
@@ -0,0 +1,313 @@
+/*
+ * Licensed under the Apache License, Version
Github user goldmedal commented on the issue:
https://github.com/apache/spark/pull/18875
@HyukjinKwon OK, I'll work on R and Python. My JIRA id is 'goldmedal', too.
Thanks for your review :)
@viirya Thanks for your mentor and review :)
---
--
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18875
BTW, would you mind if I ask your JIRA id @goldmedal? I would like to
assign it to you.
---
-
To unsubscribe, e-mail: review
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18875
Merged to master.
This was a big work for the very first contribution. Thanks for working on
this and bearing with me @goldmedal and @viirya.
Would you like to work on R and Pyt
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/18875
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18317
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81694/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18317
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/18317
**[Test build #81694 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81694/testReport)**
for PR 18317 at commit
[`f30117e`](https://github.com/apache/spark/commit/f
Github user goldmedal commented on a diff in the pull request:
https://github.com/apache/spark/pull/18875#discussion_r138501335
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonGenerator.scala
---
@@ -26,20 +26,50 @@ import
org.apache.spark.sql.cata
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19211
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/19211
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81693/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19180
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/19211
**[Test build #81693 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81693/testReport)**
for PR 19211 at commit
[`4a6858f`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19180
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81692/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19180
**[Test build #81692 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81692/testReport)**
for PR 19180 at commit
[`70c9423`](https://github.com/apache/spark/commit/7
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/18875#discussion_r138500447
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonGenerator.scala
---
@@ -26,20 +26,50 @@ import
org.apache.spark.sql.ca
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/18875#discussion_r138500520
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/jsonExpressions.scala
---
@@ -617,6 +618,14 @@ case class JsonToStructs
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19212
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81695/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19212
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/19212
**[Test build #81695 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81695/testReport)**
for PR 19212 at commit
[`80d0268`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18731
Build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18731
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81696/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18731
**[Test build #81696 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81696/testReport)**
for PR 18731 at commit
[`a53404e`](https://github.com/apache/spark/commit/a
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16422#discussion_r138498310
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -626,6 +624,74 @@ case class DescribeTableCommand(
}
Github user caneGuy commented on the issue:
https://github.com/apache/spark/pull/19132
@jerryshao @HyukjinKwon Jenkins done.Thanks too much.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For addi
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/19188
ok, I'll update in a day! Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-ma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18887
**[Test build #81697 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81697/testReport)**
for PR 18887 at commit
[`9020184`](https://github.com/apache/spark/commit/90
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18887#discussion_r138491445
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -422,208 +455,101 @@ private[history] class FsHistoryProvider(co
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18887#discussion_r138491191
--- Diff: core/src/main/scala/org/apache/spark/deploy/history/config.scala
---
@@ -0,0 +1,46 @@
+/*
+ * Licensed to the Apache Software Foundation (A
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18887#discussion_r138491139
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -742,53 +698,146 @@ private[history] object FsHistoryProvider {
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18887#discussion_r138490678
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -742,53 +698,146 @@ private[history] object FsHistoryProvider {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18659
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81691/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18659
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/18659
**[Test build #81691 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81691/testReport)**
for PR 18659 at commit
[`4a2fec2`](https://github.com/apache/spark/commit/4
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18887#discussion_r138490176
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -422,208 +455,101 @@ private[history] class FsHistoryProvider(co
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18887#discussion_r138489215
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -301,9 +334,13 @@ private[history] class FsHistoryProvider(conf:
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18887#discussion_r138488740
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -117,17 +122,37 @@ private[history] class FsHistoryProvider(conf
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18731
**[Test build #81696 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81696/testReport)**
for PR 18731 at commit
[`a53404e`](https://github.com/apache/spark/commit/a5
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18266#discussion_r138473560
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JDBCRDD.scala
---
@@ -80,7 +80,7 @@ object JDBCRDD extends Logging {
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18266#discussion_r138475068
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JDBCRelation.scala
---
@@ -111,7 +111,16 @@ private[sql] case class JD
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18266#discussion_r138476182
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcUtils.scala
---
@@ -768,6 +769,35 @@ object JdbcUtils extends Logg
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18266#discussion_r138464925
--- Diff: examples/src/main/python/sql/datasource.py ---
@@ -177,6 +177,16 @@ def jdbc_dataset_example(spark):
.jdbc("jdbc:postgresql:dbserve
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18266#discussion_r138475397
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcUtils.scala
---
@@ -768,6 +769,35 @@ object JdbcUtils extends Logg
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19212
**[Test build #81695 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81695/testReport)**
for PR 19212 at commit
[`80d0268`](https://github.com/apache/spark/commit/80
GitHub user joseph-torres opened a pull request:
https://github.com/apache/spark/pull/19212
[SPARK-21988] Add default stats to StreamingExecutionRelation.
## What changes were proposed in this pull request?
Add default stats to StreamingExecutionRelation.
## How was
Github user sitalkedia commented on a diff in the pull request:
https://github.com/apache/spark/pull/18317#discussion_r138473231
--- Diff: core/src/main/java/org/apache/spark/io/ReadAheadInputStream.java
---
@@ -0,0 +1,315 @@
+/*
+ * Licensed under the Apache License, Versi
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18317
**[Test build #81694 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81694/testReport)**
for PR 18317 at commit
[`f30117e`](https://github.com/apache/spark/commit/f3
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19211
**[Test build #81693 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81693/testReport)**
for PR 19211 at commit
[`4a6858f`](https://github.com/apache/spark/commit/4a
GitHub user vanzin opened a pull request:
https://github.com/apache/spark/pull/19211
[SPARK-18838][core] Add separate listener queues to LiveListenerBus.
This change modifies the live listener bus so that all listeners are
added to queues; each queue has its own thread to dispatc
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19180
**[Test build #81692 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81692/testReport)**
for PR 19180 at commit
[`70c9423`](https://github.com/apache/spark/commit/70
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19180
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.
Github user original-brownbear commented on the issue:
https://github.com/apache/spark/pull/19180
@srowen thanks, all comments addressed I think :)
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
F
Github user original-brownbear commented on a diff in the pull request:
https://github.com/apache/spark/pull/19180#discussion_r138466080
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ---
@@ -1097,8 +1101,21 @@ public UTF8String copy() {
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19180#discussion_r138460636
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ---
@@ -1097,8 +1101,21 @@ public UTF8String copy() {
@Override
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/19180#discussion_r138454402
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ---
@@ -1097,8 +1101,21 @@ public UTF8String copy() {
@Override
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/19180#discussion_r138453195
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ---
@@ -1097,8 +1101,21 @@ public UTF8String copy() {
@Override
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/19180#discussion_r138453256
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ---
@@ -1097,8 +1101,21 @@ public UTF8String copy() {
@Override
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/19180#discussion_r138454686
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ---
@@ -1097,8 +1101,21 @@ public UTF8String copy() {
@Override
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18659
**[Test build #81691 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81691/testReport)**
for PR 18659 at commit
[`4a2fec2`](https://github.com/apache/spark/commit/4a
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/19106
That's my sentiment too, that it is likely patching over another problem.
I'd go with Joseph on this, but at least, issue a warning maybe?
I also would support throwing an exception to fail c
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19201
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19201
Thanks! Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19201
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81689/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19201
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/19201
**[Test build #81689 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81689/testReport)**
for PR 19201 at commit
[`d456876`](https://github.com/apache/spark/commit/d
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/18659
Thanks @cloud-fan. @ueshin , it would be great to incorporate the tests
you have, let me know what is easiest for you. I'll push an update to change
to `@pandas_udf` so hopefully the tests wou
Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/18659#discussion_r138447017
--- Diff: python/pyspark/sql/functions.py ---
@@ -2112,7 +2113,7 @@ def wrapper(*args):
@since(1.3)
-def udf(f=None, returnType=Stri
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19202
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81688/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19202
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/19202
**[Test build #81688 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81688/testReport)**
for PR 19202 at commit
[`b08fd93`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19210
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user alexmnyc opened a pull request:
https://github.com/apache/spark/pull/19210
Fix Graphite re-connects for Graphite instances behind ELB's or auto-scaled
load balancers
â¦resolve hosts behind a CNAME with re-tried DNS lookups
Upgrade codahale metrics library so th
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19197
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/18317#discussion_r138430322
--- Diff: core/src/main/java/org/apache/spark/io/ReadAheadInputStream.java
---
@@ -0,0 +1,315 @@
+/*
+ * Licensed under the Apache License, Version
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/19197
having trouble merging...will try again soon
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/19197
LGTM
Merging with master and branch-2.2
Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user ptkool closed the pull request at:
https://github.com/apache/spark/pull/19209
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
GitHub user ptkool opened a pull request:
https://github.com/apache/spark/pull/19209
Branch 2.2 udf nullability
## What changes were proposed in this pull request?
When registering a Python UDF, a user may know whether the function can
return null values or not. PythonUDF a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18875
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/18875
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81684/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18875
**[Test build #81684 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81684/testReport)**
for PR 18875 at commit
[`bddf283`](https://github.com/apache/spark/commit/b
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/18592
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/18592
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.a
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19188
This has been merged. @maropu Could you update this PR? Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18592
Thanks! Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18592
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.ap
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18592
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81682/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18592
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/18592
**[Test build #81682 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81682/testReport)**
for PR 18592 at commit
[`d2d22d4`](https://github.com/apache/spark/commit/d
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/19106
Ok, I guess I'm surprised that someone even noticed this...
So, basically, we are changing the behavior of a private function for a
specific case which is actually impossible to ever run into
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19196
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81690/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19196
**[Test build #81690 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81690/testReport)**
for PR 19196 at commit
[`c5b7f23`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19196
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/19106
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/19106
**[Test build #81687 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81687/testReport)**
for PR 19106 at commit
[`53891ed`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19106
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81687/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19196
**[Test build #81690 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81690/testReport)**
for PR 19196 at commit
[`c5b7f23`](https://github.com/apache/spark/commit/c5
Github user brkyvz commented on a diff in the pull request:
https://github.com/apache/spark/pull/19196#discussion_r138414658
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/statefulOperators.scala
---
@@ -53,6 +53,8 @@ case class StatefulOperatorStateInf
Github user brkyvz commented on a diff in the pull request:
https://github.com/apache/spark/pull/19196#discussion_r138414592
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/IncrementalExecution.scala
---
@@ -117,8 +119,33 @@ class IncrementalExecution(
Github user original-brownbear commented on the issue:
https://github.com/apache/spark/pull/19180
@srowen got a sec to take a look at this one? :) (should be a quick one)
---
-
To unsubscribe, e-mail: reviews-unsubsc
101 - 200 of 436 matches
Mail list logo