You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "Josh Elser (JIRA)" <ji...@apache.org> on 2014/08/18 07:48:18 UTC

[jira] [Commented] (ACCUMULO-3005) Convert the monitor into a real REST server

    [ https://issues.apache.org/jira/browse/ACCUMULO-3005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14100304#comment-14100304 ] 

Josh Elser commented on ACCUMULO-3005:
--------------------------------------

I was playing around with this some more tonight. Made a "client POJOs" artifact and split up the server into its own shaded jar (server-side only, doesn't rebundle hadoop/zk/accumulo classes). This also removed a bunch of cruft in special handling in the shell scripts which was nice.

The sad part is that it appears that {{ch.qos.logback:logback-classic}} shaded in an SLF4J impl which prints a lovely warning every time you initialize the logging infrastructure because of the binding we expect it to use provided by slf4j-log4j12 we already have on the classpath. As sad as this is, I don't think there's a way to work around it. It's looking like I'll have to bite the bullet and just roll jetty/jersey/jackson on my own and just lift the "nice" bits from dropwizard that I like. Ugh.

> Convert the monitor into a real REST server
> -------------------------------------------
>
>                 Key: ACCUMULO-3005
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-3005
>             Project: Accumulo
>          Issue Type: Sub-task
>          Components: monitor
>            Reporter: Josh Elser
>            Assignee: Josh Elser
>             Fix For: 1.7.0
>
>         Attachments: 0001-ACCUMULO-3005-Initial-round-of-changes-for-a-proper-.patch
>
>
> The monitor works well for what it does, but it's very much so cobbled together. HTML is hand generated. Static state is used liberally which makes testing near impossible. View and controller logic are tightly intertwined which also adds to the testability problem.
> I've been working on porting the functionality from the existing monitor to a proper rest service using [dropwizard|https://dropwizard.github.io/dropwizard/]. It's not completely functional yet, but it has a number of improvements of the existing monitor code.
> I thought I would post what I have already and let people give some feedback if they're interested. Help is always welcome. https://github.com/joshelser/accumulo/tree/dropwizard.



--
This message was sent by Atlassian JIRA
(v6.2#6252)