EdColeman commented on PR #3118: URL: https://github.com/apache/accumulo/pull/3118#issuecomment-1352262349
My reasoning behind the "ERROR..." was to make it stand out and be clear that this would be an issue if not addressed and an upgrade would be attempted. The others, I'm less sure. Providing the actual perm string is sufficient to reason about what is going on - however, pulling out and specifically declaring if other can modify a node and which nodes are "public" vs Accumulo private seems convenient and makes grep expressions easier to construct for filtering. With the samples generated from an uno instance, some of this may not be as obvious, but if there were other users (say to support metrics) it seems possible that providing a summary of the three states could help to quickly review the permissions -- 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]
