[ https://issues.apache.org/jira/browse/HIVE-18575?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16354201#comment-16354201 ]
Eugene Koifman commented on HIVE-18575: --------------------------------------- what was the point of renaming isAcid to isFullAcid? {{isAcid}} was used everywhere to mean full acid table. it now leads to code like "compBuilder.setIsAcid(AcidUtils.isTransactionalTable(t));" which will cause confusion? > ACID properties usage in jobconf is ambiguous for MM tables > ----------------------------------------------------------- > > Key: HIVE-18575 > URL: https://issues.apache.org/jira/browse/HIVE-18575 > Project: Hive > Issue Type: Bug > Components: Transactions > Reporter: Sergey Shelukhin > Assignee: Sergey Shelukhin > Priority: Major > Attachments: HIVE-18575.01.patch, HIVE-18575.patch > > > Vectorization checks for ACID table trigger for MM tables where they don't > apply. Other places seem to set the setting for transactional case while most > of the code seems to assume it implies full acid. > Overall, many places in the code use the settings directly or set the ACID > flag without setting the ACID properties. -- This message was sent by Atlassian JIRA (v7.6.3#76005)