[ https://issues.apache.org/jira/browse/ACCUMULO-1778?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13795730#comment-13795730 ]
Christopher Tubbs commented on ACCUMULO-1778: --------------------------------------------- I'm not sure I like the idea of exposing details of server configuration through the public API when it isn't needed for client operations. I'm not sure client code should have any knowledge that any monitor services even exist. At the very least, I'm concerned about this being a part of the public API. What if we replace the monitor with better monitoring solutions? What if the user isn't running a monitor? What if it is running multiple monitors? Would a utility class be better suited to this? Or maybe just put it in HdfsZooInstance rather than Instance itself? Will adding it to the Instance interface disrupt the API in undesirable ways? Is there a precedent for exposing a monitoring service within the service being monitored? It seems a bit awkward for a service to assume it's being monitored at all, never mind that it's monitored with a very specific type of monitoring tool (in this case, one that has a location attribute that it shares with the services it is monitoring). > Make monitor location retrievable from Instance > ----------------------------------------------- > > Key: ACCUMULO-1778 > URL: https://issues.apache.org/jira/browse/ACCUMULO-1778 > Project: Accumulo > Issue Type: Improvement > Components: client > Reporter: Billie Rinaldi > Assignee: Billie Rinaldi > Fix For: 1.6.0 > > Attachments: ACCUMULO-1778.patch > > > If the monitor is started on a random port, it would be convenient to > retrieve the location from Instance instead of having to look it up in > Zookeeper. -- This message was sent by Atlassian JIRA (v6.1#6144)