hive-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alan Gates (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HIVE-14813) Make TransactionBatchImpl.toString() include state of each txn: commit/abort
Date Thu, 21 Sep 2017 04:17:00 GMT

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

Alan Gates commented on HIVE-14813:
-----------------------------------

If you have to go to the object level anyway why not use an enum that will be meaningful to
maintainers rather than overloading true/false/null values?  Not crucial, just curious.

+1, even if you don't decide to switch the array.

> Make TransactionBatchImpl.toString() include state of each txn: commit/abort
> ----------------------------------------------------------------------------
>
>                 Key: HIVE-14813
>                 URL: https://issues.apache.org/jira/browse/HIVE-14813
>             Project: Hive
>          Issue Type: Bug
>          Components: HCatalog, Transactions
>    Affects Versions: 1.2.1
>            Reporter: Eugene Koifman
>            Assignee: Eugene Koifman
>         Attachments: HIVE-14813.01.patch
>
>
> perhaps include user abort vs system abort on batch close
> also include if close() was called and if there was an exception
> cc [~deepesh]



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Mime
View raw message