[DISCUSSION]Remove the call to update the serde properties in case of alter scenarios

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[DISCUSSION]Remove the call to update the serde properties in case of alter scenarios

akashnilugal@gmail.com
Hi Community,

Currently, we have an API called alterTable() in
CarbonSessionCatalogUtil.scala which basically fires a SQL which goes and
changes the serde properties. Actually this API was added when carbon
needed to support alter features(spark-2.1 and 2.2 back then), but spark
wasn't supporting.

Now since we do not support old spark version and spark provides other APIs
to update schema after alter operations. So I think we can remove this API
and which makes unnecessary calls to metastore. Please give inputs on this.

Thanks,

Regards,
Akash R Nilugal
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSSION]Remove the call to update the serde properties in case of alter scenarios

David CaiQiang
+1 for removing it.



-----
Best Regards
David Cai
--
Sent from: http://apache-carbondata-dev-mailing-list-archive.1130556.n5.nabble.com/
Best Regards
David Cai