[ 
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)

Reply via email to