hvanhovell commented on a change in pull request #23249: [SPARK-26297][SQL]
improve the doc of Distribution/Partitioning
URL: https://github.com/apache/spark/pull/23249#discussion_r240268238
##
File path:
hvanhovell commented on a change in pull request #23249: [SPARK-26297][SQL]
improve the doc of Distribution/Partitioning
URL: https://github.com/apache/spark/pull/23249#discussion_r240266617
##
File path:
SparkQA commented on issue #23276: [SPARK-26321][SQL] Split a SQL in correct way
URL: https://github.com/apache/spark/pull/23276#issuecomment-445863091
**[Test build #99923 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99923/testReport)**
for PR 23276 at
AdolphKK commented on issue #23248: [SPARK-26293][SQL] Cast exception when
having python udf in subquery
URL: https://github.com/apache/spark/pull/23248#issuecomment-445858370
looks good for me, +1 :+1:
This is an automated
AdolphKK commented on issue #23277: [SPARK-26327][SQL] Metrics in
FileSourceScanExec not update correctly
URL: https://github.com/apache/spark/pull/23277#issuecomment-445857989
+1, thank you
This is an automated message from
hepin1989 commented on a change in pull request #23276: [SPARK-26321][SQL]
Split a SQL in correct way
URL: https://github.com/apache/spark/pull/23276#discussion_r240256699
##
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/StringUtils.scala
##
AmplabJenkins removed a comment on issue #23277: [SPARK-26327][SQL] Metrics in
FileSourceScanExec not update correctly
URL: https://github.com/apache/spark/pull/23277#issuecomment-445856550
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
AmplabJenkins removed a comment on issue #23277: [SPARK-26327][SQL] Metrics in
FileSourceScanExec not update correctly
URL: https://github.com/apache/spark/pull/23277#issuecomment-445856539
Merged build finished. Test PASSed.
AmplabJenkins commented on issue #23277: [SPARK-26327][SQL] Metrics in
FileSourceScanExec not update correctly
URL: https://github.com/apache/spark/pull/23277#issuecomment-445856550
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
AmplabJenkins commented on issue #23277: [SPARK-26327][SQL] Metrics in
FileSourceScanExec not update correctly
URL: https://github.com/apache/spark/pull/23277#issuecomment-445856539
Merged build finished. Test PASSed.
This
SparkQA commented on issue #23277: [SPARK-26327][SQL] Metrics in
FileSourceScanExec not update correctly
URL: https://github.com/apache/spark/pull/23277#issuecomment-445854780
**[Test build #99922 has
xuanyuanking opened a new pull request #23277: [SPARK-26327][SQL] Metrics in
FileSourceScanExec not update correctly
URL: https://github.com/apache/spark/pull/23277
## What changes were proposed in this pull request?
As the description in
mgaido91 commented on a change in pull request #23275: [SPARK-26323][SQL] Scala
UDF should still check input types even if some inputs are of type Any
URL: https://github.com/apache/spark/pull/23275#discussion_r240250904
##
File path:
AmplabJenkins removed a comment on issue #23276: [SPARK-26321][SQL] Split a SQL
in a correct way
URL: https://github.com/apache/spark/pull/23276#issuecomment-445851955
Can one of the admins verify this patch?
This is an
SparkQA commented on issue #23276: [SPARK-26321][SQL] Split a SQL in a correct
way
URL: https://github.com/apache/spark/pull/23276#issuecomment-445852714
**[Test build #99921 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99921/testReport)**
for PR 23276
AmplabJenkins removed a comment on issue #23276: [SPARK-26321][SQL] Split a SQL
in a correct way
URL: https://github.com/apache/spark/pull/23276#issuecomment-445851779
Can one of the admins verify this patch?
This is an
AmplabJenkins commented on issue #23276: [SPARK-26321][SQL] Split a SQL in a
correct way
URL: https://github.com/apache/spark/pull/23276#issuecomment-445851955
Can one of the admins verify this patch?
This is an automated
AmplabJenkins commented on issue #23276: [SPARK-26321][SQL] Split a SQL in a
correct way
URL: https://github.com/apache/spark/pull/23276#issuecomment-445851779
Can one of the admins verify this patch?
This is an automated
AmplabJenkins removed a comment on issue #23262: [SPARK-26312][SQL]Replace
RDDConversions.rowToRowRdd with RowEncoder to improve its conversion performance
URL: https://github.com/apache/spark/pull/23262#issuecomment-445849957
Test FAILed.
Refer to this link for build results (access
sadhen opened a new pull request #23276: [SPARK-26321][SQL] Split a SQL in a
correct way
URL: https://github.com/apache/spark/pull/23276
## What changes were proposed in this pull request?
Split a SQL in a correct way
## How was this patch tested?
Manually tested:
```
AmplabJenkins commented on issue #23262: [SPARK-26312][SQL]Replace
RDDConversions.rowToRowRdd with RowEncoder to improve its conversion performance
URL: https://github.com/apache/spark/pull/23262#issuecomment-445849946
Merged build finished. Test FAILed.
AmplabJenkins removed a comment on issue #23262: [SPARK-26312][SQL]Replace
RDDConversions.rowToRowRdd with RowEncoder to improve its conversion performance
URL: https://github.com/apache/spark/pull/23262#issuecomment-445849946
Merged build finished. Test FAILed.
AmplabJenkins commented on issue #23262: [SPARK-26312][SQL]Replace
RDDConversions.rowToRowRdd with RowEncoder to improve its conversion performance
URL: https://github.com/apache/spark/pull/23262#issuecomment-445849957
Test FAILed.
Refer to this link for build results (access rights to
SparkQA commented on issue #23262: [SPARK-26312][SQL]Replace
RDDConversions.rowToRowRdd with RowEncoder to improve its conversion performance
URL: https://github.com/apache/spark/pull/23262#issuecomment-445849646
**[Test build #99916 has
SparkQA removed a comment on issue #23262: [SPARK-26312][SQL]Replace
RDDConversions.rowToRowRdd with RowEncoder to improve its conversion performance
URL: https://github.com/apache/spark/pull/23262#issuecomment-445815543
**[Test build #99916 has
AmplabJenkins removed a comment on issue #23262: [SPARK-26312][SQL]Replace
RDDConversions.rowToRowRdd with RowEncoder to improve its conversion performance
URL: https://github.com/apache/spark/pull/23262#issuecomment-445846357
Test FAILed.
Refer to this link for build results (access
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23262
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/23262
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99911/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23262
**[Test build #99911 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99911/testReport)**
for PR 23262 at commit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22683
In this case the change is simpler to understand in prose, I think; "100
KB" becomes "97.6 KiB", etc.
---
-
To unsubscribe,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23275
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23275
**[Test build #99920 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99920/testReport)**
for PR 23275 at commit
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/23275#discussion_r240234573
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ScalaUDF.scala
---
@@ -47,25 +47,13 @@ case class ScalaUDF(
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/23228
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/23275#discussion_r240234371
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ScalaUDF.scala
---
@@ -47,25 +47,13 @@ case class ScalaUDF(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/23275#discussion_r240234583
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/expressions/UserDefinedFunction.scala
---
@@ -88,68 +88,49 @@ sealed trait UserDefinedFunction {
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23228
thanks, merging 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/23275
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/23275
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23275
**[Test build #99919 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99919/testReport)**
for PR 23275 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/23275#discussion_r240231883
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -4255,11 +4255,11 @@ object functions {
*
* @group udf_funcs
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/23268
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/23226
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23275
cc @maryannxue @gatorsmile @srowen
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/23275#discussion_r240230970
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ScalaUDF.scala
---
@@ -47,25 +47,13 @@ case class ScalaUDF(
GitHub user cloud-fan opened a pull request:
https://github.com/apache/spark/pull/23275
[SPARK-26323][SQL] Scala UDF should still check input types even if some
inputs are of type Any
## What changes were proposed in this pull request?
For Scala UDF, when checking input
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23226
Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23268
Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23270
**[Test build #99918 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99918/testReport)**
for PR 23270 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23270
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/23270
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23270
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23262
**[Test build #99910 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99910/testReport)**
for PR 23262 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23272
**[Test build #99914 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99914/testReport)**
for PR 23272 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23262
**[Test build #99911 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99911/testReport)**
for PR 23262 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23272
**[Test build #99915 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99915/testReport)**
for PR 23272 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23274
**[Test build #99917 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99917/testReport)**
for PR 23274 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23274
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/23274
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gaborgsomogyi commented on a diff in the pull request:
https://github.com/apache/spark/pull/23274#discussion_r240216147
--- Diff: docs/structured-streaming-kafka-integration.md ---
@@ -642,9 +642,9 @@ This way the application can be configured via Spark
parameters and
Github user gaborgsomogyi commented on a diff in the pull request:
https://github.com/apache/spark/pull/23274#discussion_r240215906
--- Diff: docs/structured-streaming-kafka-integration.md ---
@@ -642,9 +642,9 @@ This way the application can be configured via Spark
parameters and
GitHub user gaborgsomogyi opened a pull request:
https://github.com/apache/spark/pull/23274
[SPARK-26322][SS] Add spark.kafka.token.sasl.mechanism to ease delegation
token configuration.
## What changes were proposed in this pull request?
When Kafka delegation token
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23249
**[Test build #99912 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99912/testReport)**
for PR 23249 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23262
**[Test build #99913 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99913/testReport)**
for PR 23262 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23262
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/23262
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/23262
**[Test build #99916 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99916/testReport)**
for PR 23262 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23262
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/23249
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/23249
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/23262
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/23262
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/23272
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/23262
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/23262
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/23262
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/23272
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/23272
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/23272
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/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/99906/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22957
**[Test build #99906 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99906/testReport)**
for PR 22957 at commit
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/23262
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/23272#discussion_r240204508
--- Diff:
core/src/test/java/org/apache/spark/unsafe/map/AbstractBytesToBytesMapSuite.java
---
@@ -667,4 +669,54 @@ public void testPeakMemoryUsed() {
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/23272#discussion_r240204245
--- Diff:
core/src/test/java/org/apache/spark/unsafe/map/AbstractBytesToBytesMapSuite.java
---
@@ -667,4 +669,54 @@ public void testPeakMemoryUsed() {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23228
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/23228
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99902/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23228
**[Test build #99902 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99902/testReport)**
for PR 23228 at commit
Github user eatoncys commented on the issue:
https://github.com/apache/spark/pull/23262
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/23272#discussion_r240189245
--- Diff:
core/src/test/java/org/apache/spark/unsafe/map/AbstractBytesToBytesMapSuite.java
---
@@ -667,4 +669,54 @@ public void testPeakMemoryUsed() {
Github user eatoncys commented on the issue:
https://github.com/apache/spark/pull/23262
@cloud-fan Updated, thanks.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/23272#discussion_r240187707
--- Diff:
core/src/test/java/org/apache/spark/memory/TestMemoryConsumer.java ---
@@ -38,12 +38,14 @@ public long spill(long size, MemoryConsumer trigger)
Github user eatoncys commented on a diff in the pull request:
https://github.com/apache/spark/pull/23262#discussion_r240188043
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSourceStrategy.scala
---
@@ -416,7 +416,12 @@ case class
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/23272#discussion_r240187678
--- Diff:
core/src/test/java/org/apache/spark/unsafe/map/AbstractBytesToBytesMapSuite.java
---
@@ -667,4 +668,53 @@ public void testPeakMemoryUsed() {
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/23272
> have you seen any bug report caused by this dead lock?
The original reporter of the JIRA ticket SPARK-26265 has hit with this bug
in their workload.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23272
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/23272
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99900/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23272
**[Test build #99900 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99900/testReport)**
for PR 23272 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23262
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/23262
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99903/
Test PASSed.
---
601 - 700 of 826 matches
Mail list logo