Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22815#discussion_r228594291
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala ---
@@ -54,6 +54,7 @@ import org.apache.spark.sql.util.ExecutionListenerManager
* @groupname Ungrouped Support functions for language integrated queries
* @since 1.0.0
*/
+@deprecated("Use SparkSession instead", "3.0.0")
--- End diff --
Yea I wouldn't deprecate it now ... the data source API v1 still depends on
it.
Actually now I think about it, we should not be deprecate SQLContext until
dsv2 is stable. Otherwise we have a stable API dsv1 depending on a deprecated
API.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
