Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16595
**[Test build #71427 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71427/testReport)**
for PR 16595 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16592
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71420/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16592
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16592
**[Test build #71420 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71420/testReport)**
for PR 16592 at commit
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/16566#discussion_r9617
--- Diff: R/pkg/R/mllib_clustering.R ---
@@ -38,6 +45,146 @@ setClass("KMeansModel", representation(jobj = "jobj"))
#' @note LDAModel since 2.1.0
GitHub user yanboliang opened a pull request:
https://github.com/apache/spark/pull/16595
[Minor][YARN] Move YarnSchedulerBackendSuite to resource-managers/yarn
directory.
## What changes were proposed in this pull request?
#16092 moves YARN resource manager related code to
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/16561
LGTM
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/16561
**[Test build #71426 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71426/testReport)**
for PR 16561 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16561#discussion_r96175295
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Cast.scala
---
@@ -89,6 +89,25 @@ object Cast {
case _ => false
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16474
---
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/16387
**[Test build #71425 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71425/testReport)**
for PR 16387 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/16474
thanks, merging to master/2.1!
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/16594
**[Test build #71424 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71424/testReport)**
for PR 16594 at commit
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/16594
cc @rxin @cloud-fan
---
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 wzhfy opened a pull request:
https://github.com/apache/spark/pull/16594
[SPARK-17078] [SQL] Show stats when explain
## What changes were proposed in this pull request?
Currently we can only check the estimated stats in logical plans by
debugging. We need to
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/16387
@samkum Thanks for testing this. I think it is because every time
`forceSpill` is called now, it will spill the map anyway. I will add a check to
only spill the map if the map is not empty.
---
If
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16474
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71417/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16474
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16474
**[Test build #71417 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71417/testReport)**
for PR 16474 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16308#discussion_r96171888
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CastSuite.scala
---
@@ -316,30 +329,43 @@ class CastSuite extends
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/16308#discussion_r96171901
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/finishAnalysis.scala
---
@@ -41,13 +46,18 @@ object ReplaceExpressions extends
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16308#discussion_r96171760
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CastSuite.scala
---
@@ -32,20 +34,20 @@ import
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16593
**[Test build #71423 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71423/testReport)**
for PR 16593 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16308#discussion_r96171393
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala
---
@@ -195,19 +231,26 @@ case class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16591
**[Test build #71422 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71422/testReport)**
for PR 16591 at commit
Github user uncleGen commented on the issue:
https://github.com/apache/spark/pull/16591
This work does not change any code, but just delete unused imports and fix
some code style issue.
cc @srowen
---
If your project is set up for it, you can reply to this email and have your
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16308#discussion_r96171142
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/finishAnalysis.scala
---
@@ -41,13 +46,18 @@ object ReplaceExpressions
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16593
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16593
**[Test build #71421 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71421/testReport)**
for PR 16593 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16593
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71421/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16593
**[Test build #71421 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71421/testReport)**
for PR 16593 at commit
GitHub user windpiger opened a pull request:
https://github.com/apache/spark/pull/16593
[SPARK-19153][SQL]DataFrameWriter.saveAsTable work with create partitioned
table
## What changes were proposed in this pull request?
After
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/16474
LGTM
---
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 jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16561#discussion_r96168755
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/view.scala
---
@@ -28,22 +28,56 @@ import
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16464
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71418/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16464
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16464
**[Test build #71418 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71418/testReport)**
for PR 16464 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16561#discussion_r96168453
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/view.scala
---
@@ -28,22 +28,56 @@ import
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16561#discussion_r96168416
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/view.scala
---
@@ -28,22 +28,56 @@ import
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16561#discussion_r96168283
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/view.scala
---
@@ -28,22 +28,56 @@ import
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16561#discussion_r96168225
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/view.scala
---
@@ -28,22 +28,56 @@ import
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16561#discussion_r96167779
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/view.scala
---
@@ -28,22 +28,56 @@ import
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16561
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71416/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16561
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16561
**[Test build #71416 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71416/testReport)**
for PR 16561 at commit
Github user samkum commented on the issue:
https://github.com/apache/spark/pull/16387
I have tested this, but I found a very strange observation. GC frequency
has increased many folds...and majority of the time is spend in GC.
---
If your project is set up for it, you can reply to
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16592
cc @cloud-fan I think we really need to do this ASAP for improving the test
case coverage in DDL commands, when I do the PR:
https://github.com/apache/spark/pull/16587
---
If your project is
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/16344
Jenkins, test 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
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16592#discussion_r96166442
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -102,6 +76,198 @@ class DDLSuite extends QueryTest with
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16592
**[Test build #71420 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71420/testReport)**
for PR 16592 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16592#discussion_r96166347
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -102,6 +76,198 @@ class DDLSuite extends QueryTest with
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/16592
[SPARK-19235] [SQL] [TESTS] Enable Test Cases in DDLSuite with Hive
Metastore
### What changes were proposed in this pull request?
So far, the test cases in DDLSuites only verify the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12064
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71419/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12064
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12064
**[Test build #71419 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71419/testReport)**
for PR 12064 at commit
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/16344
add to whitelist.
---
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
Github user wangmiao1981 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16566#discussion_r96165457
--- Diff: R/pkg/R/mllib_clustering.R ---
@@ -38,6 +45,146 @@ setClass("KMeansModel", representation(jobj = "jobj"))
#' @note LDAModel since 2.1.0
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/16344
ok to test.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12064
**[Test build #71419 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71419/testReport)**
for PR 12064 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16464
**[Test build #71418 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71418/testReport)**
for PR 16464 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16588
Thanks!
---
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
Github user gatorsmile closed the pull request at:
https://github.com/apache/spark/pull/16588
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16474
**[Test build #71417 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71417/testReport)**
for PR 16474 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16561
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16561
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71415/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16561
**[Test build #71415 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71415/testReport)**
for PR 16561 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16591
I know this is acceptable assuming from the history. However, I have seen a
lot of unused imports across the code base. I think it'd be nicer if the same
instances are checked at least within
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16591
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16591
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71414/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16591
**[Test build #71414 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71414/testReport)**
for PR 16591 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16545
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
enabled
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16545
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71413/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16545
**[Test build #71413 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71413/testReport)**
for PR 16545 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16474#discussion_r96162649
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileScanRDD.scala
---
@@ -135,11 +135,21 @@ class FileScanRDD(
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16474#discussion_r96162528
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileScanRDD.scala
---
@@ -135,11 +135,21 @@ class FileScanRDD(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15505
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71412/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15505
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15505
**[Test build #71412 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71412/testReport)**
for PR 15505 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16561
**[Test build #71416 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71416/testReport)**
for PR 16561 at commit
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16561#discussion_r96159390
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala
---
@@ -198,9 +203,44 @@ case class CatalogTable(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16209
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71410/
Test PASSed.
---
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16517#discussion_r96158823
--- Diff:
core/src/main/scala/org/apache/spark/internal/io/HadoopMapReduceCommitProtocol.scala
---
@@ -99,7 +99,7 @@ class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16209
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16209
**[Test build #71410 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71410/testReport)**
for PR 16209 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16517#discussion_r96158740
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/InsertIntoHiveTable.scala
---
@@ -69,34 +69,31 @@ import
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/16588
thanks, merging to 2.1
---
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
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16587#discussion_r96158395
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalogSuite.scala
---
@@ -278,7 +277,7 @@ abstract class
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16587#discussion_r96158356
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/InMemoryCatalog.scala
---
@@ -186,6 +186,11 @@ class InMemoryCatalog(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16561#discussion_r96158153
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/view.scala
---
@@ -28,22 +28,56 @@ import
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16561#discussion_r96157931
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLViewSuite.scala
---
@@ -680,21 +700,70 @@ class SQLViewSuite extends QueryTest
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16561#discussion_r96157834
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLViewSuite.scala
---
@@ -680,21 +700,70 @@ class SQLViewSuite extends QueryTest
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16561#discussion_r96157384
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala
---
@@ -198,9 +203,44 @@ case class CatalogTable(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16561#discussion_r96157302
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala
---
@@ -198,9 +203,44 @@ case class CatalogTable(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16561#discussion_r96157263
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala
---
@@ -254,6 +294,9 @@ case class CatalogTable(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16561#discussion_r96157156
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala
---
@@ -198,9 +203,44 @@ case class CatalogTable(
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/16528
cc @yhuai for final sign-off
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16561
**[Test build #71415 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71415/testReport)**
for PR 16561 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16591
**[Test build #71414 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71414/testReport)**
for PR 16591 at commit
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16587#discussion_r96155690
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -1692,20 +1678,27 @@ class DDLSuite extends QueryTest
GitHub user uncleGen opened a pull request:
https://github.com/apache/spark/pull/16591
[SPARK-19227][CORE] remove ununsed imports and outdated comments in
`org.apache.spark.internal.config.ConfigEntry`
## What changes were proposed in this pull request?
remove ununsed imports
1 - 100 of 246 matches
Mail list logo