[ https://issues.apache.org/jira/browse/HIVE-26217?focusedWorklogId=773317&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-773317 ]
ASF GitHub Bot logged work on HIVE-26217: ----------------------------------------- Author: ASF GitHub Bot Created on: 23/May/22 07:25 Start Date: 23/May/22 07:25 Worklog Time Spent: 10m Work Description: deniskuzZ commented on code in PR #3281: URL: https://github.com/apache/hive/pull/3281#discussion_r879104158 ########## ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java: ########## @@ -7592,6 +7594,22 @@ protected Operator genFileSinkPlan(String dest, QB qb, Operator input) destTableIsTransactional = tblProps != null && AcidUtils.isTablePropertyTransactional(tblProps); if (destTableIsTransactional) { + isNonNativeTable = MetaStoreUtils.isNonNativeTable(tblProps); + boolean isCtas = tblDesc != null && tblDesc.isCTAS(); + isMmTable = isMmCreate = AcidUtils.isInsertOnlyTable(tblProps); + if (!isNonNativeTable && !destTableIsTemporary && isCtas) { + destTableIsFullAcid = AcidUtils.isFullAcidTable(tblProps); + acidOperation = getAcidType(dest); + isDirectInsert = isDirectInsert(destTableIsFullAcid, acidOperation); + boolean enableSuffixing = conf.getBoolVar(ConfVars.HIVE_ACID_CREATE_TABLE_USE_SUFFIX) Review Comment: nit, please use HiveConf getter: ```` HiveConf.getBoolVar(conf, ConfVars.HIVE_ACID_CREATE_TABLE_USE_SUFFIX) ```` Issue Time Tracking ------------------- Worklog Id: (was: 773317) Time Spent: 2h 40m (was: 2.5h) > Make CTAS use Direct Insert Semantics > ------------------------------------- > > Key: HIVE-26217 > URL: https://issues.apache.org/jira/browse/HIVE-26217 > Project: Hive > Issue Type: Improvement > Reporter: Sourabh Badhya > Assignee: Sourabh Badhya > Priority: Major > Labels: pull-request-available > Time Spent: 2h 40m > Remaining Estimate: 0h > > CTAS on transactional tables currently does a copy from staging location to > table location. This can be avoided by using Direct Insert semantics. Added > support for suffixed table locations as well. -- This message was sent by Atlassian Jira (v8.20.7#820007)