Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21100
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/21123
**[Test build #89764 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89764/testReport)**
for PR 21123 at commit
[`f940088`](https://github.com/apache/spark/commit/f9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21100
**[Test build #89755 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89755/testReport)**
for PR 21100 at commit
[`670824f`](https://github.com/apache/spark/commit/6
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21130
LGTM except for a tiny typo.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: r
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21137
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/21137
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89760/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21137
**[Test build #89760 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89760/testReport)**
for PR 21137 at commit
[`b39b6cd`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20146
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/20146
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/2611/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20146
**[Test build #89763 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89763/testReport)**
for PR 20146 at commit
[`a1dcfda`](https://github.com/apache/spark/commit/a1
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20146#discussion_r183619017
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/StringIndexer.scala ---
@@ -130,21 +159,57 @@ class StringIndexer @Since("1.4.0") (
@Since(
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21130#discussion_r183616570
--- Diff: R/pkg/R/functions.R ---
@@ -189,6 +189,11 @@ NULL
#' the map or array of maps.
#' \item \code{from_json}: it is the
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/20692
@mgaido91 Can you update this? Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/20938
@ueshin Do you forget to merge this? ;)
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21113
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/21113
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89753/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21113
**[Test build #89753 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89753/testReport)**
for PR 21113 at commit
[`c8adec6`](https://github.com/apache/spark/commit/c
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/21018#discussion_r183614108
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryRelation.scala
---
@@ -55,56 +42,39 @@ object InMemoryRelation {
priv
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21018
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/2610/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21018
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/21018
**[Test build #89762 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89762/testReport)**
for PR 21018 at commit
[`c17c5fb`](https://github.com/apache/spark/commit/c1
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20937
Seems fine but please allow me to take another look, which I will take
within this weekend.
---
-
To unsubscribe, e-mail: re
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20146
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/2609/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20146
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21100#discussion_r183611758
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -896,6 +896,25 @@ class SQLQuerySuite extends QueryTest with
SharedSQLC
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20146
**[Test build #89761 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89761/testReport)**
for PR 20146 at commit
[`ed35d87`](https://github.com/apache/spark/commit/ed
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/21123#discussion_r183611332
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/sources/v2/FileDataSourceV2Suite.scala
---
@@ -0,0 +1,179 @@
+/*
+ * Licensed to the A
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/21123#discussion_r183611071
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/sources/v2/FileDataSourceV2Suite.scala
---
@@ -0,0 +1,179 @@
+/*
+ * Licensed to the A
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20146
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21137
@hvanhovell
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spar
Github user liutang123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21100#discussion_r183608838
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -896,6 +896,25 @@ class SQLQuerySuite extends QueryTest with
SharedSQL
Github user Ngone51 commented on the issue:
https://github.com/apache/spark/pull/20998
Agree and thank you @squito .
And thanks for all of you. @felixcheung @mridulm @jiangxb1987 @srowen
---
-
To unsubscrib
Github user jose-torres commented on the issue:
https://github.com/apache/spark/pull/21135
LGTM, I think it's broadly correct for query nodes to assume the session
has been initialized
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21137
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/2608/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21137
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user jose-torres commented on the issue:
https://github.com/apache/spark/pull/21136
LGTM. TypedFilter and Filter share the FilterExec execution node, so this
should just work.
Ideally we would add a test to ContinuousSuite to ensure that TypedFilter
does execute proper
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21137
**[Test build #89760 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89760/testReport)**
for PR 21137 at commit
[`b39b6cd`](https://github.com/apache/spark/commit/b3
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/21126#discussion_r183605577
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/StreamingQuerySuite.scala
---
@@ -492,6 +492,77 @@ class StreamingQuerySuite extends S
GitHub user maropu opened a pull request:
https://github.com/apache/spark/pull/21137
[SPARK-23589][SQL][FOLLOW-UP] Reuse InternalRow in ExternalMapToCatalyst
eval
## What changes were proposed in this pull request?
This pr is a follow-up of #20980 and fixes code to reuse `Intern
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21070
Can we run a TPCDS and show that this upgrade doesn't cause performance
regression in Spark? I can see that this new version doesn't have perf
regression at parquet side, just want to be sure the
Github user xuanyuanking commented on the issue:
https://github.com/apache/spark/pull/21136
+1 for this.
We find this by CP app use filter with functions, this can be supported by
current implement.
cc @jose-torres @zsxwing @tdas
---
-
Github user xuanyuanking commented on a diff in the pull request:
https://github.com/apache/spark/pull/21136#discussion_r183604217
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/UnsupportedOperationsSuite.scala
---
@@ -771,7 +778,16 @@ class Unsupport
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/21126#discussion_r183604136
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/ProgressReporter.scala
---
@@ -207,62 +209,92 @@ trait ProgressReporter exte
Github user yanlin-Lynn commented on the issue:
https://github.com/apache/spark/pull/21136
@xuanyuanking , please help to review for this path. Thank you!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apach
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21136
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/21136
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user yanlin-Lynn opened a pull request:
https://github.com/apache/spark/pull/21136
[SPARK-24061][SS]Add TypedFilter support for continuous processing
## What changes were proposed in this pull request?
Add TypedFilter support for continuous processing application.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20937
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/20937
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89751/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20937
**[Test build #89751 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89751/testReport)**
for PR 20937 at commit
[`a7be182`](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/21100#discussion_r183602450
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -896,6 +896,25 @@ class SQLQuerySuite extends QueryTest with
SharedSQLC
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21100#discussion_r183601150
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -896,6 +896,25 @@ class SQLQuerySuite extends QueryTest with
SharedSQL
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21100#discussion_r183601024
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -896,6 +896,25 @@ class SQLQuerySuite extends QueryTest with
SharedSQLC
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21078
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89754/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21078
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/21078
**[Test build #89754 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89754/testReport)**
for PR 21078 at commit
[`492dc46`](https://github.com/apache/spark/commit/4
Github user bersprockets commented on a diff in the pull request:
https://github.com/apache/spark/pull/21073#discussion_r183600236
--- Diff: python/pyspark/sql/functions.py ---
@@ -2186,6 +2186,29 @@ def map_values(col):
return Column(sc._jvm.functions.map_values(_to_java_c
Github user bersprockets commented on a diff in the pull request:
https://github.com/apache/spark/pull/21073#discussion_r18365
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CollectionExpressionsSuite.scala
---
@@ -56,6 +58,26 @@ class Collecti
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21073
**[Test build #89759 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89759/testReport)**
for PR 21073 at commit
[`13baf96`](https://github.com/apache/spark/commit/13
Github user caneGuy closed the pull request at:
https://github.com/apache/spark/pull/21127
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user caneGuy commented on the issue:
https://github.com/apache/spark/pull/21127
Ok i will close this pr.
Thanks for your time @srowen @vanzin
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20923
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/20923
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89748/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20923
**[Test build #89748 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89748/testReport)**
for PR 20923 at commit
[`f6b9dc8`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21135
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/21135
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user pwoody commented on the issue:
https://github.com/apache/spark/pull/21135
@ericl @gatorsmile @jose-torres
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mai
GitHub user pwoody opened a pull request:
https://github.com/apache/spark/pull/21135
[SPARK-24060][TEST] StreamingSymmetricHashJoinHelperSuite should initialize
after SparkSession creation
## What changes were proposed in this pull request?
We should ensure that the SparkSession
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21100
**[Test build #89757 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89757/testReport)**
for PR 21100 at commit
[`8cb240f`](https://github.com/apache/spark/commit/8c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20940
**[Test build #89758 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89758/testReport)**
for PR 20940 at commit
[`8ae0126`](https://github.com/apache/spark/commit/8a
Github user squito commented on the issue:
https://github.com/apache/spark/pull/20940
Jenkins, retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: re
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21129
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/2607/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21129
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/21129
**[Test build #89756 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89756/testReport)**
for PR 21129 at commit
[`170e08f`](https://github.com/apache/spark/commit/17
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/21127
Version info is already available, in the code and in the UI. "Compiled by"
info has never struck me as useful. The rest is from the env. I don't think
this adds anything.
---
Github user caneGuy commented on the issue:
https://github.com/apache/spark/pull/21127
How about the other information?
As mentioned,the build info @vanzin
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20633
**[Test build #4156 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4156/testReport)**
for PR 20633 at commit
[`80b668a`](https://github.com/apache/spark/commit/
Github user caneGuy closed the pull request at:
https://github.com/apache/spark/pull/21128
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user caneGuy commented on the issue:
https://github.com/apache/spark/pull/21128
Got it,thanks @vanzin
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21116
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/21116
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89747/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21116
**[Test build #89747 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89747/testReport)**
for PR 21116 at commit
[`b676dc8`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21100
**[Test build #89755 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89755/testReport)**
for PR 21100 at commit
[`670824f`](https://github.com/apache/spark/commit/67
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21078
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/2606/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21078
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/21078
**[Test build #89754 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89754/testReport)**
for PR 21078 at commit
[`492dc46`](https://github.com/apache/spark/commit/49
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20940
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89746/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20940
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/20940
**[Test build #89746 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89746/testReport)**
for PR 20940 at commit
[`8ae0126`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21113
**[Test build #89753 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89753/testReport)**
for PR 21113 at commit
[`c8adec6`](https://github.com/apache/spark/commit/c8
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21123#discussion_r183589976
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/sources/v2/FileDataSourceV2Suite.scala
---
@@ -0,0 +1,179 @@
+/*
+ * Licensed to the Apach
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21123#discussion_r183589841
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/sources/v2/FileDataSourceV2Suite.scala
---
@@ -0,0 +1,179 @@
+/*
+ * Licensed to the Apach
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21123#discussion_r183589906
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/sources/v2/FileDataSourceV2Suite.scala
---
@@ -0,0 +1,179 @@
+/*
+ * Licensed to the Apach
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21123#discussion_r183589673
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/sources/v2/FileDataSourceV2Suite.scala
---
@@ -0,0 +1,179 @@
+/*
+ * Licensed to the Apach
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21123#discussion_r183589528
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/sources/v2/FileDataSourceV2Suite.scala
---
@@ -0,0 +1,179 @@
+/*
+ * Licensed to the Apach
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21123#discussion_r183589427
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/sources/v2/FileDataSourceV2Suite.scala
---
@@ -0,0 +1,179 @@
+/*
+ * Licensed to the Apach
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21123#discussion_r183589226
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/sources/v2/FileDataSourceV2Suite.scala
---
@@ -0,0 +1,179 @@
+/*
+ * Licensed to the Apach
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21123#discussion_r183589146
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/BaseSessionStateBuilder.scala
---
@@ -158,6 +158,7 @@ abstract class BaseSessionStateBuil
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21072
---
-
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/21072
thanks, merging to master!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
1 - 100 of 569 matches
Mail list logo
