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

Eugene Koifman commented on HIVE-13249:
---------------------------------------

I think  maxOpenTxns numOpenTxns  tooManyOpenTxns should all be volatile since 
they are accessed by different threads
I would make LOG.warn("Transaction database not properly configured ... be 
"error"
You don't need to add "OpenTxnsCounter" - nothing bad can happen if 
AcidOpenTxnsCounterService runs in parallel - it's just a count(*) query
Because TxnHandler exists in standalone metastore (e.g. HS2), you probably need 
to start AcidOpenTxnsCounterService from TxnHandler.setConf() but such that 
there is only 1 instance of AcidOpenTxnsCounterService per JVM

> Hard upper bound on number of open transactions
> -----------------------------------------------
>
>                 Key: HIVE-13249
>                 URL: https://issues.apache.org/jira/browse/HIVE-13249
>             Project: Hive
>          Issue Type: Improvement
>          Components: Transactions
>    Affects Versions: 2.0.0
>            Reporter: Wei Zheng
>            Assignee: Wei Zheng
>         Attachments: HIVE-13249.1.patch, HIVE-13249.2.patch, 
> HIVE-13249.3.patch, HIVE-13249.4.patch, HIVE-13249.5.patch, 
> HIVE-13249.6.patch, HIVE-13249.7.patch
>
>
> We need to have a safeguard by adding an upper bound for open transactions to 
> avoid huge number of open-transaction requests, usually due to improper 
> configuration of clients such as Storm.
> Once that limit is reached, clients will start failing.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to