veghlaci05 opened a new pull request, #4566:
URL: https://github.com/apache/hive/pull/4566
### What changes were proposed in this pull request?
Standardization and simplification of JDBC related code inTxnHandler
Key changes:
Standardized transaction handling
Standardized Retry and exception handling
No direct connection handling
Use of RowMapper/ResultSetExtractor/RowCallbackHandler interfaces for DB
<-> DAO class mapping
Extract code parts from TxnHandler: Ideally these classes should contain
the business logic only, and the infrastructure related code parts
(transaction, connection, retry handling, object mapping, etc) should be
extracted into individual classes.
### Why are the changes needed?
TxnHandler is getting too complex, and a source of tech-debt accumulation
### Does this PR introduce _any_ user-facing change?
No
### Is the change a dependency upgrade?
No
### How was this patch tested?
Through unit tests
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
