dlmarion commented on code in PR #5350:
URL: https://github.com/apache/accumulo/pull/5350#discussion_r1967811855
##########
server/base/src/main/java/org/apache/accumulo/server/conf/store/impl/PropStoreWatcher.java:
##########
@@ -100,15 +103,15 @@ public void process(final WatchedEvent event) {
case NodeDataChanged:
path = event.getPath();
log.trace("handle change event for path: {}", path);
- propStoreKey = PropStoreKey.fromPath(path);
+ propStoreKey = PropStoreKey.fromPath(path, instanceId);
Review Comment:
I'm curious about the event.path here. Is it the full path with the
`/accumulo/<instanceId>` or is it relative?
##########
server/base/src/main/java/org/apache/accumulo/server/tables/TableManager.java:
##########
@@ -260,7 +252,7 @@ public void accept(WatchedEvent event) {
final String zPath = event.getPath();
Review Comment:
I have the same question here about the Watcher even path being full or
relative.
--
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]
