Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19110
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19110
**[Test build #81352 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81352/testReport)**
for PR 19110 at commit
[`24f4499`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19110
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81352/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19082
**[Test build #81351 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81351/testReport)**
for PR 19082 at commit
[`a2b3dd3`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19082
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81351/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19082
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/19082
Jenkins, retest this please.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wis
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19082
**[Test build #81354 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81354/testReport)**
for PR 19082 at commit
[`a2b3dd3`](https://github.com/apache/spark/commit/a2
GitHub user felixcheung opened a pull request:
https://github.com/apache/spark/pull/19111
[SPARK-21801][SPARKR][TEST][WIP] set random seed for predictable test
## What changes were proposed in this pull request?
set.seed() before running tests
## How was this patch
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19111
**[Test build #81355 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81355/testReport)**
for PR 19111 at commit
[`1857a1c`](https://github.com/apache/spark/commit/18
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19018
#19111
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19111
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19111
**[Test build #81355 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81355/testReport)**
for PR 19111 at commit
[`1857a1c`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19111
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81355/
Test FAILed.
---
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/18931#discussion_r136711183
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ExpandExec.scala ---
@@ -89,6 +89,8 @@ case class ExpandExec(
child.asInstanceOf[Co
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/18931#discussion_r136710872
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/WholeStageCodegenExec.scala
---
@@ -149,14 +149,146 @@ trait CodegenSupport extends SparkPl
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/18931#discussion_r136710741
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/WholeStageCodegenExec.scala
---
@@ -149,14 +149,146 @@ trait CodegenSupport extends SparkPl
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/18931#discussion_r136710667
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/WholeStageCodegenExec.scala
---
@@ -149,14 +149,146 @@ trait CodegenSupport extends SparkPl
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/18931#discussion_r136710909
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/WholeStageCodegenExec.scala
---
@@ -149,14 +149,146 @@ trait CodegenSupport extends SparkPl
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/18931#discussion_r136710824
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/WholeStageCodegenExec.scala
---
@@ -149,14 +149,146 @@ trait CodegenSupport extends SparkPl
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/18931#discussion_r136710234
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/WholeStageCodegenExec.scala
---
@@ -149,14 +149,146 @@ trait CodegenSupport extends SparkPl
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/18931#discussion_r136711099
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SortExec.scala ---
@@ -177,6 +177,8 @@ case class SortExec(
""".stripMargin.trim
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/18931#discussion_r136710928
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/WholeStageCodegenExec.scala
---
@@ -149,14 +149,146 @@ trait CodegenSupport extends SparkPl
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19109#discussion_r136711660
--- Diff: docs/index.md ---
@@ -24,7 +24,7 @@ source, visit [Building Spark](building-spark.html).
Spark runs on both Windows and UNIX-like sy
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19082
**[Test build #81354 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81354/testReport)**
for PR 19082 at commit
[`a2b3dd3`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19082
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81354/
Test PASSed.
---
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/19082
@viirya @rednaxelafx @kiszk okay, could you check again?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19082
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18869
**[Test build #81356 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81356/testReport)**
for PR 18869 at commit
[`ec9199a`](https://github.com/apache/spark/commit/ec
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18865
**[Test build #81357 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81357/testReport)**
for PR 18865 at commit
[`b397cb3`](https://github.com/apache/spark/commit/b3
Github user jmchung commented on the issue:
https://github.com/apache/spark/pull/18865
To @viirya and @gatorsmile, I made some modifications as follows:
1. move the check of `_corrupt_record` out of the function block to get
fast fail in driver instead of executor side.
2. `sp
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/18865#discussion_r136714700
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JsonFileFormat.scala
---
@@ -113,6 +113,18 @@ class JsonFileFormat ex
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/18865#discussion_r136714509
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JsonFileFormat.scala
---
@@ -113,6 +113,18 @@ class JsonFileFormat ex
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/18865#discussion_r136714872
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JsonFileFormat.scala
---
@@ -113,6 +113,18 @@ class JsonFileFormat ex
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/18865#discussion_r136714892
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -2034,4 +2034,26 @@ class JsonSuite extends Qu
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/18865#discussion_r136714668
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JsonFileFormat.scala
---
@@ -113,6 +113,18 @@ class JsonFileFormat ex
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19105
Thank you @viirya and @felixcheung.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
ena
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19105
Merged to master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19105
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is ena
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19105
I am hitting a network issue. Will update JIRA tomorrow.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user yssharma commented on the issue:
https://github.com/apache/spark/pull/18029
Resolved conflict introduced by other code commits.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have thi
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18029
**[Test build #81358 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81358/testReport)**
for PR 18029 at commit
[`0c46008`](https://github.com/apache/spark/commit/0c
Github user jmchung commented on a diff in the pull request:
https://github.com/apache/spark/pull/18865#discussion_r136715367
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JsonFileFormat.scala
---
@@ -113,6 +113,18 @@ class JsonFileFormat extend
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18869
**[Test build #81356 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81356/testReport)**
for PR 18869 at commit
[`ec9199a`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18869
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81356/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18869
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18029
**[Test build #81358 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81358/testReport)**
for PR 18029 at commit
[`0c46008`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18029
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81358/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18029
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18869
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes s
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18865
@HyukjinKwon Thanks for the comment.
I think the current behavior confuses users in some ways, as it can have
weird query results shown in previous discussion.
The previous fix that
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18869
**[Test build #81359 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81359/testReport)**
for PR 18869 at commit
[`ec9199a`](https://github.com/apache/spark/commit/ec
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18865
**[Test build #81357 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81357/testReport)**
for PR 18865 at commit
[`b397cb3`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18865
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18865
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81357/
Test PASSed.
---
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18865
@gatorsmile and @cloud-fan, do you guys prefer throwing an exception or
printing a log? I think I like logging one more conservatively.
BTW, I am sorry for raising this issue late now.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18865
**[Test build #81360 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81360/testReport)**
for PR 18865 at commit
[`da61df3`](https://github.com/apache/spark/commit/da
Github user cenyuhai commented on the issue:
https://github.com/apache/spark/pull/17924
@dongjoon-hyun I have a question: does this orc data sources reader support
a table contains multiple file format
for example:
table/
day=2017-09-01 RCFile
day=2017-0
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18869
**[Test build #81359 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81359/testReport)**
for PR 18869 at commit
[`ec9199a`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18869
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/17924
Hi, I didn't try that, but that's not a concept of Spark data source table.
Please don't expect that. :)
---
If your project is set up for it, you can reply to this email and have your
reply
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/17924
BTW, the latest version is maintained in #17980.
Recently, Spark Vector format is changed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18869
retest this please.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, o
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18869
**[Test build #81362 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81362/testReport)**
for PR 18869 at commit
[`ec9199a`](https://github.com/apache/spark/commit/ec
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18869
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81359/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19086
**[Test build #81361 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81361/testReport)**
for PR 19086 at commit
[`cd00825`](https://github.com/apache/spark/commit/cd
Github user jinxing64 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19086#discussion_r136719337
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -502,17 +502,16 @@ private[spark] class HiveExternalCatalog(c
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16774#discussion_r136719383
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tuning/CrossValidator.scala ---
@@ -100,31 +113,53 @@ class CrossValidator @Since("1.2.0") (@Since("
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16774#discussion_r136719561
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tuning/TrainValidationSplit.scala ---
@@ -87,37 +91,63 @@ class TrainValidationSplit @Since("1.5.0")
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16774#discussion_r136719485
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tuning/CrossValidator.scala ---
@@ -100,31 +113,53 @@ class CrossValidator @Since("1.2.0") (@Since("
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18865
`requiredSchema.length == 1 && requiredSchema.head.name ==
parsedOptions.columnNameOfCorruptRecord`
What is the usage scenario if we do not block the above scenario?
---
If your projec
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18865
**[Test build #81360 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81360/testReport)**
for PR 18865 at commit
[`da61df3`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18865
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18865
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81360/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19086
**[Test build #81361 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81361/testReport)**
for PR 19086 at commit
[`cd00825`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19086
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19086
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81361/
Test FAILed.
---
Github user demoonism commented on the issue:
https://github.com/apache/spark/pull/19109
Closing as the change is subtle.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user demoonism closed the pull request at:
https://github.com/apache/spark/pull/19109
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18869
**[Test build #81362 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81362/testReport)**
for PR 18869 at commit
[`ec9199a`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18869
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18869
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81362/
Test PASSed.
---
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/18953
Hi, All.
Although ORC seems not to be a prefered storage format in Apache Spark, ORC
is very important to me. Could anyone review this again?
---
If your project is set up for it, you can
GitHub user jaceklaskowski opened a pull request:
https://github.com/apache/spark/pull/19112
[SPARK-21901][SS] Define toString for StateOperatorProgress
## What changes were proposed in this pull request?
Just `StateOperatorProgress.toString` + few formatting fixes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19112
**[Test build #81363 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81363/testReport)**
for PR 19112 at commit
[`337ad48`](https://github.com/apache/spark/commit/33
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19111
**[Test build #81364 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81364/testReport)**
for PR 19111 at commit
[`5d156be`](https://github.com/apache/spark/commit/5d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19060
**[Test build #81365 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81365/testReport)**
for PR 19060 at commit
[`fb72c89`](https://github.com/apache/spark/commit/fb
Github user janewangfb commented on a diff in the pull request:
https://github.com/apache/spark/pull/18975#discussion_r136724143
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
---
@@ -178,11 +179,50 @@ class AstBuilder(conf: SQLConf) ex
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/19060
Hi, @gatorsmile .
I believe I understand your advice correctly at this time.
Could you take a look at this `data source` verification PR?
---
If your project is set up for it, you can
Github user janewangfb commented on a diff in the pull request:
https://github.com/apache/spark/pull/18975#discussion_r136724258
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
---
@@ -178,11 +179,50 @@ class AstBuilder(conf: SQLConf) ex
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/19112#discussion_r136724317
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/streaming/progress.scala ---
@@ -200,7 +202,7 @@ class SourceProgress protected[sql](
*/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18975
**[Test build #81366 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81366/testReport)**
for PR 18975 at commit
[`2ec9947`](https://github.com/apache/spark/commit/2e
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/19112#discussion_r136724364
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/streaming/progress.scala ---
@@ -177,11 +179,11 @@ class SourceProgress protected[sql](
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19111
**[Test build #81364 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81364/testReport)**
for PR 19111 at commit
[`5d156be`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19111
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19111
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81364/
Test PASSed.
---
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19086#discussion_r136724927
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalogSuite.scala
---
@@ -418,6 +436,39 @@ abstract class SessionCat
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19086#discussion_r136723544
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -573,28 +573,29 @@ class SessionCatalog(
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19086#discussion_r136724919
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalogSuite.scala
---
@@ -418,6 +436,39 @@ abstract class SessionCat
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19086#discussion_r136724969
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -502,17 +502,16 @@ private[spark] class HiveExternalCatalog(
1 - 100 of 231 matches
Mail list logo