-
- Downloads
[SPARK-15552][SQL] Remove unnecessary private[sql] methods in SparkSession
## What changes were proposed in this pull request? SparkSession has a list of unnecessary private[sql] methods. These methods cause some trouble because private[sql] doesn't apply in Java. In the cases that they are easy to remove, we can simply remove them. This patch does that. As part of this pull request, I also replaced a bunch of protected[sql] with private[sql], to tighten up visibility. ## How was this patch tested? Updated test cases to reflect the changes. Author: Reynold Xin <rxin@databricks.com> Closes #13319 from rxin/SPARK-15552.
Showing
- sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala 2 additions, 2 deletions...src/main/scala/org/apache/spark/sql/DataFrameWriter.scala
- sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala 16 additions, 16 deletionssql/core/src/main/scala/org/apache/spark/sql/Dataset.scala
- sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala 6 additions, 43 deletions...core/src/main/scala/org/apache/spark/sql/SQLContext.scala
- sql/core/src/main/scala/org/apache/spark/sql/SparkSession.scala 21 additions, 33 deletions...re/src/main/scala/org/apache/spark/sql/SparkSession.scala
- sql/core/src/main/scala/org/apache/spark/sql/execution/CacheManager.scala 1 addition, 1 deletion...n/scala/org/apache/spark/sql/execution/CacheManager.scala
- sql/core/src/main/scala/org/apache/spark/sql/execution/QueryExecution.scala 1 addition, 1 deletion...scala/org/apache/spark/sql/execution/QueryExecution.scala
- sql/core/src/main/scala/org/apache/spark/sql/execution/command/commands.scala 1 addition, 1 deletion...ala/org/apache/spark/sql/execution/command/commands.scala
- sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala 2 additions, 1 deletion...in/scala/org/apache/spark/sql/execution/command/ddl.scala
- sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala 3 additions, 3 deletions.../scala/org/apache/spark/sql/execution/command/views.scala
- sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala 1 addition, 1 deletion...g/apache/spark/sql/execution/datasources/DataSource.scala
- sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/InsertIntoDataSourceCommand.scala 1 addition, 1 deletion...l/execution/datasources/InsertIntoDataSourceCommand.scala
- sql/core/src/main/scala/org/apache/spark/sql/internal/CatalogImpl.scala 12 additions, 12 deletions...ain/scala/org/apache/spark/sql/internal/CatalogImpl.scala
- sql/core/src/main/scala/org/apache/spark/sql/internal/SessionState.scala 3 additions, 1 deletion...in/scala/org/apache/spark/sql/internal/SessionState.scala
- sql/core/src/test/scala/org/apache/spark/sql/CachedTableSuite.scala 8 additions, 8 deletions...rc/test/scala/org/apache/spark/sql/CachedTableSuite.scala
- sql/core/src/test/scala/org/apache/spark/sql/DataFramePivotSuite.scala 1 addition, 1 deletion...test/scala/org/apache/spark/sql/DataFramePivotSuite.scala
- sql/core/src/test/scala/org/apache/spark/sql/DatasetCacheSuite.scala 7 additions, 5 deletions...c/test/scala/org/apache/spark/sql/DatasetCacheSuite.scala
- sql/core/src/test/scala/org/apache/spark/sql/JoinSuite.scala 4 additions, 4 deletionssql/core/src/test/scala/org/apache/spark/sql/JoinSuite.scala
- sql/core/src/test/scala/org/apache/spark/sql/execution/columnar/InMemoryColumnarQuerySuite.scala 4 additions, 4 deletions...k/sql/execution/columnar/InMemoryColumnarQuerySuite.scala
- sql/core/src/test/scala/org/apache/spark/sql/execution/metric/SQLMetricsSuite.scala 10 additions, 8 deletions...g/apache/spark/sql/execution/metric/SQLMetricsSuite.scala
- sql/core/src/test/scala/org/apache/spark/sql/execution/ui/SQLListenerSuite.scala 8 additions, 8 deletions.../org/apache/spark/sql/execution/ui/SQLListenerSuite.scala
Loading
Please register or sign in to comment