wchevreuil commented on PR #4418: URL: https://github.com/apache/hbase/pull/4418#issuecomment-1149790005
Have been going back and forth with @BukrosSzabolcs on this, discussing what would be alternatives for a more efficient solution. Unfortunately, our conclusion is that unless we can implement some sort of "global" registry of referenced mob files, there's not much left we can do. Such refactoring would be more suitable under a whole MOB redesign. So for now, I think we are good to go with this PR, which main objective is to avoid usage of temp dirs/renames when FILE SFT is enabled, making MOB behaviour aligned with the conventional regions behaviour. Regarding the latest UT failures, these should actually be addressed by HBASE-27017 commit. I'm good to approve this once we rebase it and get a green UT run. Thanks for being super patient, @BukrosSzabolcs ! -- 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]
