cshannon commented on issue #4857: URL: https://github.com/apache/accumulo/issues/4857#issuecomment-2363884856
I started to look at this now that the changes with #4852 were merged and it looks relatively straight forward. One thing I noticed, which might be better as a separate PR first because it also would affect 3.1, is the lack of validation for the directory column. I think it would be nice to add a check and new violation if that format is wrong. There is already a utility to do the validation: `ServerColumnFamily.validateDirCol()` I can work on a quick PR for that and test improvements, and another PR to use the opid in elasticity to detect splits. -- 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]
