ctubbsii commented on pull request #1958: URL: https://github.com/apache/accumulo/pull/1958#issuecomment-792976585
> `ZooLock.validateAndSortChildrenByLockPrefix` validates and sorts the locks at the associated path for ZooLock's purposes. Right. The problem here is that there was a coding error that incorrectly interpreted this lock path as being one where ZooLock's locks are stored... but that shouldn't have been the case. I don't think we should merely log... we should actively prevent such coding errors from occurring by avoiding String-based parameters in ZooLock (and ZooQueueLock) APIs, where possible. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
