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

Eugene Koifman commented on HIVE-19876:
---------------------------------------

+1 patch 6.

Incidentally, acid (i.e. DbTxnManager always uses DbLockManager) ignores 

hive.lock.manager setting so 

isValidTxnListState() does not need any logic for other LMs.  This is just FYI, 
for future.

> Multiple fixes for Driver.isValidTxnListState
> ---------------------------------------------
>
>                 Key: HIVE-19876
>                 URL: https://issues.apache.org/jira/browse/HIVE-19876
>             Project: Hive
>          Issue Type: Bug
>          Components: Transactions
>    Affects Versions: 3.1.0, 4.0.0
>            Reporter: Eugene Koifman
>            Assignee: Jesus Camacho Rodriguez
>            Priority: Major
>         Attachments: HIVE-19876.06.patch
>
>
> 1) Only locks for tables should be checked.
> 2) Check on whether {{txnWriteIdList}} is null or not needs to be fixed: it 
> should be done on the configuration property value, as the object 
> {{txnWriteIdList}} will never be null.



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

Reply via email to