dlmarion commented on pull request #1958: URL: https://github.com/apache/accumulo/pull/1958#issuecomment-792963262
`ZooLock.validateAndSortChildrenByLockPrefix` validates and sorts the locks at the associated path for ZooLock's purposes. If the ephemeral node name does not meet the format specifications of ZooLock, then maybe we should just make that a `trace` level event instead of `warn`. The list of valid children is printed at the `trace` level at the end of the method, maybe the invalid children should be the same. My suggestion applies to the log message [here](https://github.com/apache/accumulo/blob/main/core/src/main/java/org/apache/accumulo/fate/zookeeper/ZooLock.java#L215) and [here](https://github.com/apache/accumulo/blob/main/core/src/main/java/org/apache/accumulo/fate/zookeeper/ZooLock.java#L218) ---------------------------------------------------------------- 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]
