[
https://issues.apache.org/jira/browse/HIVE-12637?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15243890#comment-15243890
]
Eugene Koifman commented on HIVE-12637:
---------------------------------------
[~wzheng]
1. For the description of HIVE_TXN_RETRYABLE_SQLEX_REGEX I would add that the
string that the regex will check is "ex.getMessage() + "(SQLState=" +
ex.getSQLState() + ",ErrorCode=" + ex.getErrorCode() + ")"" where "ex" is a
SQLException. I think this would make it more intuitive for end users.
2. testRetryableRegex() could you add a test where the error msg includes a ","?
> make retryable SQLExceptions in TxnHandler configurable
> -------------------------------------------------------
>
> Key: HIVE-12637
> URL: https://issues.apache.org/jira/browse/HIVE-12637
> Project: Hive
> Issue Type: Improvement
> Components: Transactions
> Affects Versions: 1.0.0
> Reporter: Eugene Koifman
> Assignee: Wei Zheng
> Attachments: HIVE-12637.1.patch
>
>
> same for CompactionTxnHandler
> would be convenient if the user could specify some RegEx (perhaps by db type)
> which will tell TxnHandler.checkRetryable() that this is should be retried.
> The regex should probably apply to String produced by
> {noformat}
> private static String getMessage(SQLException ex) {
> return ex.getMessage() + "(SQLState=" + ex.getSQLState() + ",ErrorCode="
> + ex.getErrorCode() + ")";
> }
> {noformat}
> This make it flexible.
> See if we need to add Db type (and possibly version) of the DB being used.
> With 5 different DBs supported this gives control end users.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
