HyukjinKwon commented on a change in pull request #30554:
URL: https://github.com/apache/spark/pull/30554#discussion_r533814216
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala
##########
@@ -2922,6 +2922,15 @@ object SQLConf {
.stringConf
.createWithDefault("")
+ val LEGACY_CREATE_HIVE_TABLE_BY_DEFAULT =
+ buildConf("spark.sql.legacy.createHiveTableByDefault")
+ .internal()
+ .doc("When set to true, CREATE TABLE syntax without a table provider
will use hive " +
+ s"instead of the value of ${DEFAULT_DATA_SOURCE_NAME.key} as the table
provider.")
Review comment:
There are already a lot of legacy configurations, and there's no plan
for how long we'll keep it. That's what I know as far as I have followed in the
community. It would be a separate issue to decide lifetime of legacy
configurations but ideally the legacy configurations will be removed in the
major release bumpup I guess. I remember I discussed this with Sean as well
somewhere. cc @srowen FYI
----------------------------------------------------------------
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
