[
https://issues.apache.org/jira/browse/HIVE-18124?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Eugene Koifman updated HIVE-18124:
----------------------------------
Description:
With the addition of MM tables (_AcidUtils.isInsertOnlyTable(table)_) the
methods in AcidUtils and dependent places are very muddled. There are now a
number of places where we have something like _isAcidTable =
AcidUtils.isFullAcidTable(table)_ and a later getter
_boolean isAcidTable() \{ return isAcidTable;\}_
Need to clean it up so that there is a isTransactional(Table) that checks
transactional=true setting and isAcid(Table) to mean full ACID and
isInsertOnly(Table) to mean MM tables.
This would accurately describe the semantics of the tables.
was:
With the addition of MM tables (_AcidUtils.isInsertOnlyTable(table)_) the
methods in AcidUtils and dependent places are very muddled.
Need to clean it up so that there is a isTransactional(Table) that checks
transactional=true setting and isAcid(Table) to mean full ACID and
isInsertOnly(Table) to mean MM tables.
This would accurately describe the semantics of the tables.
> clean up isAcidTable() API vs isInsertOnlyTable()
> --------------------------------------------------
>
> Key: HIVE-18124
> URL: https://issues.apache.org/jira/browse/HIVE-18124
> Project: Hive
> Issue Type: Bug
> Components: Transactions
> Affects Versions: 3.0.0
> Reporter: Eugene Koifman
> Assignee: Eugene Koifman
> Attachments: HIVE-18124.01.patch, HIVE-18124.02.patch,
> HIVE-18124.03.patch, HIVE-18124.04.patch, HIVE-18124.05.patch
>
>
> With the addition of MM tables (_AcidUtils.isInsertOnlyTable(table)_) the
> methods in AcidUtils and dependent places are very muddled. There are now a
> number of places where we have something like _isAcidTable =
> AcidUtils.isFullAcidTable(table)_ and a later getter
> _boolean isAcidTable() \{ return isAcidTable;\}_
> Need to clean it up so that there is a isTransactional(Table) that checks
> transactional=true setting and isAcid(Table) to mean full ACID and
> isInsertOnly(Table) to mean MM tables.
> This would accurately describe the semantics of the tables.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
