accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Josh Elser (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-3005) Convert the monitor into a real REST server
Date Mon, 18 Aug 2014 05:48:18 GMT

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

Mime
View raw message