charlesconnell commented on PR #7084: URL: https://github.com/apache/hbase/pull/7084#issuecomment-2959265464
Another option, which is simpler to reason about, is to simply fail the `SnapshotProcedure` if the table isn't in a state it can handle. This was my approach in HBASE-29315, for the exact same reasons you're encountering here. I couldn't usefully "sleep" a `SplitTableRegionProcedure` because it used holdLock=true. I think it would be good to agree on a standard of what to do in these situations. Failing a procedure is simpler and doesn't introduce more edge cases in the procedure executor state machine. However, obviously, it's a better user experience if your procedures get executed eventually. -- 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]
