[jira] [Commented] (HIVE-20986) Add TransactionalValidationListener to HMS preListeners only when ACID support is enabled

["Karthik Manamcheri (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/HIVE-20986?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16733514#comment-16733514 ] 

Karthik Manamcheri commented on HIVE-20986:
-------------------------------------------

This can be alternatively addressed by HIVE-20977 instead. There is no one flag to indicate if ACID support is enabled in Hive or not. 

> Add TransactionalValidationListener to HMS preListeners only when ACID support is enabled
> -----------------------------------------------------------------------------------------
>
>                 Key: HIVE-20986
>                 URL: https://issues.apache.org/jira/browse/HIVE-20986
>             Project: Hive
>          Issue Type: Improvement
>            Reporter: Karthik Manamcheri
>            Assignee: Adam Holley
>            Priority: Major
>
> We add the TransactionalValidationListener to the preListeners in HMS unconditionally.
> {code:java}
> public void init() throws MetaException {
>   ..
>   preListeners.add(0, new TransactionalValidationListener(conf));
>   ..
> }{code}
> This causes some performance issues because the listener is called even when not needed. Lets add a condition around this and add this listener only if the transactional support is enabled.
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)