[GitHub] [hive] deniskuzZ commented on pull request #3656: [WIP]HIVE-11495: Add aborted reason to transaction information.

[GitBox <gi...@apache.org>]

deniskuzZ commented on PR #3656:
URL: https://github.com/apache/hive/pull/3656#issuecomment-1274240038

   @simhadri-g I don't think it's a good idea to populate something in TXNS table for further troubleshooting. Once a transaction is aborted and there is nothing to clean up, the corresponding row in TXNS table would be immediately deleted.  
   Note: aborted txns prevent the cleanup of ACID metadata, so we try to keep their life short. 
   I think we should try to focus on improving the logging here. However, if you want to persist abort reason in DB, maybe we need to pass the aborted txn reason through the compaction tables as well. Those could be used later on for the inspection.


-- 
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: gitbox-unsubscribe@hive.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org