pvary commented on code in PR #3015:
URL: https://github.com/apache/hive/pull/3015#discussion_r841657777
##########
ql/src/java/org/apache/hadoop/hive/ql/lockmgr/DbTxnManager.java:
##########
@@ -721,7 +721,7 @@ private void stopHeartbeat() {
// The lock should not be held by other thread trying to stop the
heartbeat for more than 31 seconds
isLockAcquired = heartbeatTaskLock.tryLock(31000, TimeUnit.MILLISECONDS);
} catch (InterruptedException e) {
- // safe to go on
+ Thread.currentThread().interrupt();
Review Comment:
How sure are we about this change?
Who will handle this Interrupt?
--
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]
