[
https://issues.apache.org/jira/browse/HIVE-24541?focusedWorklogId=530674&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-530674
]
ASF GitHub Bot logged work on HIVE-24541:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 04/Jan/21 13:56
Start Date: 04/Jan/21 13:56
Worklog Time Spent: 10m
Work Description: pvary commented on a change in pull request #1797:
URL: https://github.com/apache/hive/pull/1797#discussion_r551330709
##########
File path: common/src/java/org/apache/hadoop/hive/conf/HiveConf.java
##########
@@ -1991,6 +1991,10 @@ private static void
populateLlapDaemonVarsSet(Set<String> llapDaemonVarsSetLocal
"Default file format for CREATE TABLE statement applied to managed
tables only. External tables will be \n" +
"created with format specified by hive.default.fileformat. Leaving
this null will result in using hive.default.fileformat \n" +
"for all tables."),
+ HIVEDEFAULTSTORAGEHANDLER("hive.default.storage.handler.class", "",
Review comment:
Could be please use underscores to separate names? I understand that the
ones before and after are not using them, but currently the underscore is the
"standard"
----------------------------------------------------------------
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:
us...@infra.apache.org
Issue Time Tracking
-------------------
Worklog Id: (was: 530674)
Time Spent: 20m (was: 10m)
> Add config to set a default storage handler class
> -------------------------------------------------
>
> Key: HIVE-24541
> URL: https://issues.apache.org/jira/browse/HIVE-24541
> Project: Hive
> Issue Type: Improvement
> Reporter: Marton Bod
> Assignee: Marton Bod
> Priority: Major
> Labels: pull-request-available
> Time Spent: 20m
> Remaining Estimate: 0h
>
> Add a config param "hive.default.storage.handler.class" so we can set a
> default storage handler class that can be used for all create table
> statements. By default it would be an empty string, taking no effect.
> This would allow existing user queries to be reused for a new table format
> for example, such as Iceberg.
> For example, after setting in the config: hive.default.storage.handler.class=
> org.apache.iceberg.mr.hive.HiveIcebergStorageHandler
> The query: CREATE TABLE abc (a int, b string) LOCATION ...
> would be equivalent to: CREATE TABLE abc (a int, b string) STORED BY
> 'org.apache.iceberg.mr.hive.HiveIcebergStorageHandler' LOCATION ...
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
