ctubbsii commented on code in PR #5350:
URL: https://github.com/apache/accumulo/pull/5350#discussion_r1992475598
##########
server/base/src/main/java/org/apache/accumulo/server/conf/util/ZooPropEditor.java:
##########
@@ -180,7 +179,8 @@ private void printProperties(final ServerContext context,
final PropStoreKey<?>
writer.printf(": Property scope: %s\n", scope);
writer.printf(": ZooKeeper path: %s\n", propKey.getPath());
writer.printf(": Name: %s\n", getDisplayName(propKey, context));
- writer.printf(": Id: %s\n", propKey.getId());
+ writer.printf(": Id: %s\n", propKey instanceof IdBasedPropStoreKey
+ ? ((IdBasedPropStoreKey<?>) propKey).getId() : "N/A");
Review Comment:
There is no such method. The "N/A" is specifically for the PropKey thatis
*not* one that is id-based. So, there's no ID to print, and no canonical form
of the ID. It simply doesn't apply to PropKeys that aren't based on an ID.
--
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]
