Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23171
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23171
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23166
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
GitHub user aokolnychyi opened a pull request:
https://github.com/apache/spark/pull/23171
[SPARK-26205][SQL] Optimize In for bytes, shorts, ints
## What changes were proposed in this pull request?
This PR optimizes `In` expressions for byte, short, integer types. It is a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23136
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99392/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23136
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23136
**[Test build #99392 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99392/testReport)**
for PR 23136 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23136
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/5466/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23165
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23165
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99385/
Test FAILed.
---
Github user squito commented on the issue:
https://github.com/apache/spark/pull/23166
> The only question I have is does this cause more memory usage on the
driver because it has a reference to that broadcast value or is something else
already holding on to it?
yeah good
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23165
**[Test build #99385 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99385/testReport)**
for PR 23165 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23136
**[Test build #99392 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99392/testReport)**
for PR 23136 at commit
Github user liyinan926 commented on the issue:
https://github.com/apache/spark/pull/23136
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/23166#discussion_r237212353
--- Diff: python/pyspark/tests/test_broadcast.py ---
@@ -67,6 +67,20 @@ def test_broadcast_with_encryption(self):
def
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/23166
Yeah certainly seems like a good idea. The only question I have is does
this cause more memory usage on the driver because it has a reference to that
broadcast value or is something else already
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/23052#discussion_r237210777
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVFileFormat.scala
---
@@ -169,13 +169,18 @@ private[csv] class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22612
**[Test build #99391 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99391/testReport)**
for PR 22612 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/23124
Thank you so much, @cloud-fan !
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/22598#discussion_r237206264
--- Diff:
core/src/main/scala/org/apache/spark/deploy/security/KafkaTokenUtil.scala ---
@@ -0,0 +1,168 @@
+/*
+ * Licensed to the Apache
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/22598#discussion_r237203744
--- Diff:
core/src/main/scala/org/apache/spark/deploy/security/KafkaTokenUtil.scala ---
@@ -0,0 +1,168 @@
+/*
+ * Licensed to the Apache
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23052
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23052
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99380/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23052
**[Test build #99380 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99380/testReport)**
for PR 23052 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23170
**[Test build #99390 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99390/testReport)**
for PR 23170 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23170
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23170
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23168
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99389/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23168
**[Test build #99389 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99389/testReport)**
for PR 23168 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23168
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user DaveDeCaprio commented on the issue:
https://github.com/apache/spark/pull/23076
Closing this pull request in favor of #23169
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user DaveDeCaprio closed the pull request at:
https://github.com/apache/spark/pull/23076
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user DaveDeCaprio commented on the issue:
https://github.com/apache/spark/pull/23169
@MaxGekk and @hvanhovell, this is an alternative solution for #23076. It
limits overall plan length when generating the full string in memory, but not
if a specific writer is passed in.
---
GitHub user wangyum opened a pull request:
https://github.com/apache/spark/pull/23170
[SPARK-24423][FOLLOW-UP][SQL] Fix error example
## What changes were proposed in this pull request?
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23169
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user squito commented on the issue:
https://github.com/apache/spark/pull/23166
maybe a dumb question -- couldn't you just add
```python
self._value = value
```
[inside `__init__()` when on the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23169
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/23169
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user DaveDeCaprio opened a pull request:
https://github.com/apache/spark/pull/23169
[SPARK-26103][SQL] Limit the length of debug strings for query plans
## What changes were proposed in this pull request?
The PR puts in a limit on the size of a debug string generated
Github user huaxingao commented on the issue:
https://github.com/apache/spark/pull/23168
@felixcheung Could you please review?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/23088#discussion_r237197020
--- Diff: core/src/main/scala/org/apache/spark/status/AppStatusStore.scala
---
@@ -222,29 +223,20 @@ private[spark] class AppStatusStore(
val
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23168
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23168
**[Test build #99389 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99389/testReport)**
for PR 23168 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23168
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
GitHub user huaxingao opened a pull request:
https://github.com/apache/spark/pull/23168
[SPARK-26207][doc]add PowerIterationClustering (PIC) doc in 2.4 branch
## What changes were proposed in this pull request?
Add PIC doc in 2.4
## How was this patch tested?
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23167
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99386/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23167
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23167
**[Test build #99386 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99386/testReport)**
for PR 23167 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/22887
> So it's reasonble for users to expect that, if they set hadoop config via
the SQL SET command, it should override the one in spark-defaults.conf.
I agree with that. But the previous
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/23055#discussion_r237188935
--- Diff: python/pyspark/worker.py ---
@@ -22,7 +22,12 @@
import os
import sys
import time
-import resource
+# 'resource' is a Unix
Github user ankuriitg commented on the issue:
https://github.com/apache/spark/pull/23058
The change looks good to me. I understand that this change uses memory
efficiently but I am wondering whether it causes any performance degradation
compared to memory mapping. If yes, can we
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/23150
Looks like that kind of fallback logic was there for compatibility with
Spark 1.x and 2.0; see some comments about 'backwards compatibility' in for
example
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23150
Some tests didn't pass on new changes till I set time zone explicitly. The
tests use the same functions for checking correctness as the code that is
supposed to test. I think need more
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/23086#discussion_r237179854
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2ScanExec.scala
---
@@ -54,27 +53,17 @@ case class
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23150
I have to correct timestamp/date pattern in a few test to follow ISO 8601
(see [Patterns for Formatting and
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/23086#discussion_r237178976
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2ScanExec.scala
---
@@ -23,29 +23,28 @@ import
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23031
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23031
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23166
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23166
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23031
**[Test build #99388 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99388/testReport)**
for PR 23031 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23166
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99387/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23166
**[Test build #99387 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99387/testReport)**
for PR 23166 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23166
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/23086#discussion_r237176552
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameReader.scala ---
@@ -38,7 +38,7 @@ import org.apache.spark.sql.execution.datasources.jdbc._
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23167
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/23086#discussion_r237176100
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/TableProvider.java ---
@@ -0,0 +1,62 @@
+/*
+ * Licensed to the Apache Software
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23167
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23150
> Are there any other behavior changes with the new code, besides being
able to parse microseconds?
The main one is new parser doesn't have the fallback to
`DateTimeUtils.stringToTime` if
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23167
**[Test build #99386 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99386/testReport)**
for PR 23167 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23166
**[Test build #99387 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99387/testReport)**
for PR 23166 at commit
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/23166
cc @squito
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/23166
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
GitHub user srowen opened a pull request:
https://github.com/apache/spark/pull/23167
[SPARK-26024][FOLLOWUP][MINOR] Follow-up to remove extra blank lines in R
function descriptions
## What changes were proposed in this pull request?
Follow-up to remove extra blank lines in
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/23025#discussion_r237174135
--- Diff: R/pkg/R/DataFrame.R ---
@@ -767,6 +767,14 @@ setMethod("repartition",
#' using \code{spark.sql.shuffle.partitions} as
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23161
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23161
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99384/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23161
**[Test build #99384 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99384/testReport)**
for PR 23161 at commit
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/23086#discussion_r237172065
--- Diff: sql/core/src/main/java/org/apache/spark/sql/sources/v2/Table.java
---
@@ -0,0 +1,51 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23031
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23031
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99374/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23031
**[Test build #99374 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99374/testReport)**
for PR 23031 at commit
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/23055#discussion_r237169532
--- Diff: python/pyspark/worker.py ---
@@ -22,7 +22,12 @@
import os
import sys
import time
-import resource
+# 'resource' is a Unix
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22957
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22957
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99375/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22957
**[Test build #99375 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99375/testReport)**
for PR 22957 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22514
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22514
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99378/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22514
**[Test build #99378 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99378/testReport)**
for PR 22514 at commit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/23150
The code looks good at a glance. So the flag lets people select the old
behavior; that keeps it pretty safe. Are there any other behavior changes with
the new code, besides being able to parse
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23166
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/23166
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/23166
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/23088#discussion_r237164158
--- Diff: core/src/main/scala/org/apache/spark/status/AppStatusStore.scala
---
@@ -222,29 +223,20 @@ private[spark] class AppStatusStore(
val
GitHub user redsanket opened a pull request:
https://github.com/apache/spark/pull/23166
[SPARK-26201] Fix python broadcast with encryption
## What changes were proposed in this pull request?
Python with rpc and disk encryption enabled along with a python broadcast
variable and
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23150
@srowen @HyukjinKwon @viirya @mgaido91 May I ask you to look at the PR. The
changes are related to another PR which you have reviewed already.
---
Github user rezasafi commented on a diff in the pull request:
https://github.com/apache/spark/pull/22612#discussion_r237161182
--- Diff:
core/src/main/scala/org/apache/spark/executor/ProcfsMetricsGetter.scala ---
@@ -0,0 +1,231 @@
+/*
+ * Licensed to the Apache Software
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23165
**[Test build #99385 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99385/testReport)**
for PR 23165 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23165
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23165
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
401 - 500 of 769 matches
Mail list logo