[GitHub] [accumulo] ctubbsii commented on pull request #3727: re #3724: tablet transaction log

["ctubbsii (via GitHub)" <gi...@apache.org>]

ctubbsii commented on PR #3727:
URL: https://github.com/apache/accumulo/pull/3727#issuecomment-1711834736

   I was also planning on taking another look later today. I would prefer we hold off until next week at least to give more time to review and polish. I'm particularly interested in how close the current implementation is to the outline I made in my previous comment, and where it diverges, what are the substantive differences between the rough ideas I had in my head, and how it actually is implemented. Given that #3733 has been addressed, I don't think this is particularly urgent, but I think 2.1.3 is a reasonable target, once everybody is happy with it. So, while I think we should hold off on merging it for now, I will mark it as a blocker for 2.1.3 so it doesn't get punted past the next patch release.


-- 
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: notifications-unsubscribe@accumulo.apache.org

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