[GitHub] [carbondata] QiangCai commented on a change in pull request #3751: [CARBONDATA-3803] Mark CarbonSession as deprecated since 2.0

[GitBox <gi...@apache.org>]

QiangCai commented on a change in pull request #3751:
URL: https://github.com/apache/carbondata/pull/3751#discussion_r421958314



##########
File path: integration/spark/src/main/scala/org/apache/spark/sql/CarbonSession.scala
##########
@@ -40,12 +40,18 @@ import org.apache.carbondata.streaming.CarbonStreamingQueryListener
  * Session implementation for {org.apache.spark.sql.SparkSession}
  * Implemented this class only to use our own SQL DDL commands.
  * User needs to use {CarbonSession.getOrCreateCarbon} to create Carbon session.
+ *
+ * @deprecated Since 2.0, only use for backward compatibility,
+ *             please switch to use {@link CarbonExtensions}.
  */
+@Deprecated
 class CarbonSession(@transient val sc: SparkContext,

Review comment:
       the document already contains it




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org