-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/23988/
-----------------------------------------------------------
Review request for accumulo.
Bugs: ACCUMULO-3005
https://issues.apache.org/jira/browse/ACCUMULO-3005
Repository: accumulo
Description
-------
Creates a proper REST service using Dropwizard, with the intent to eventually
replace the existing monitor's "data" component. Copies most of the
functionality (sans the log-forwarding) into a standalone application. Returns
data as JSON and tries to separate logic into consumable pieces. Still uses the
Monitor class for most Thrift interactions.
Diffs
-----
assemble/bin/accumulo 727a4c8
assemble/bin/start-all.sh cebbd8c
assemble/bin/stop-all.sh 4bf06c0
assemble/bin/stop-server.sh 52696af
assemble/conf/templates/accumulo-site.xml 08c905b
assemble/pom.xml 89a3747
pom.xml ba6693d
server/monitor-rest/.gitignore PRE-CREATION
server/monitor-rest/pom.xml PRE-CREATION
server/monitor-rest/src/main/java/org/apache/accumulo/monitor/rest/MonitorApplication.java
PRE-CREATION
server/monitor-rest/src/main/java/org/apache/accumulo/monitor/rest/MonitorConfiguration.java
PRE-CREATION
server/monitor-rest/src/main/java/org/apache/accumulo/monitor/rest/api/GarbageCollection.java
PRE-CREATION
server/monitor-rest/src/main/java/org/apache/accumulo/monitor/rest/api/GarbageCollectorCycle.java
PRE-CREATION
server/monitor-rest/src/main/java/org/apache/accumulo/monitor/rest/api/GarbageCollectorStatus.java
PRE-CREATION
server/monitor-rest/src/main/java/org/apache/accumulo/monitor/rest/api/LogEvent.java
PRE-CREATION
server/monitor-rest/src/main/java/org/apache/accumulo/monitor/rest/api/RecoveryStatusInformation.java
PRE-CREATION
server/monitor-rest/src/main/java/org/apache/accumulo/monitor/rest/api/ReplicationInformation.java
PRE-CREATION
server/monitor-rest/src/main/java/org/apache/accumulo/monitor/rest/api/TableInformation.java
PRE-CREATION
server/monitor-rest/src/main/java/org/apache/accumulo/monitor/rest/api/TabletServerInformation.java
PRE-CREATION
server/monitor-rest/src/main/java/org/apache/accumulo/monitor/rest/api/TabletServerTableInformation.java
PRE-CREATION
server/monitor-rest/src/main/java/org/apache/accumulo/monitor/rest/api/TabletServerWithTableInformation.java
PRE-CREATION
server/monitor-rest/src/main/java/org/apache/accumulo/monitor/rest/health/AccumuloHealthCheck.java
PRE-CREATION
server/monitor-rest/src/main/java/org/apache/accumulo/monitor/rest/resources/GarbageCollectorResource.java
PRE-CREATION
server/monitor-rest/src/main/java/org/apache/accumulo/monitor/rest/resources/LogResource.java
PRE-CREATION
server/monitor-rest/src/main/java/org/apache/accumulo/monitor/rest/resources/MasterResource.java
PRE-CREATION
server/monitor-rest/src/main/java/org/apache/accumulo/monitor/rest/resources/ProblemsResource.java
PRE-CREATION
server/monitor-rest/src/main/java/org/apache/accumulo/monitor/rest/resources/ReplicationResource.java
PRE-CREATION
server/monitor-rest/src/main/java/org/apache/accumulo/monitor/rest/resources/StatisticsOverTimeResource.java
PRE-CREATION
server/monitor-rest/src/main/java/org/apache/accumulo/monitor/rest/resources/StatisticsResource.java
PRE-CREATION
server/monitor-rest/src/main/java/org/apache/accumulo/monitor/rest/resources/TablesResource.java
PRE-CREATION
server/monitor-rest/src/main/java/org/apache/accumulo/monitor/rest/resources/TabletServerResource.java
PRE-CREATION
server/monitor-rest/src/main/resources/accumulo-monitor-rest.yaml
PRE-CREATION
server/monitor-rest/src/test/resources/log4j.properties PRE-CREATION
server/monitor/src/main/java/org/apache/accumulo/monitor/Monitor.java 268516c
Diff: https://reviews.apache.org/r/23988/diff/
Testing
-------
Tested against a single-node Accumulo instance. While this code is much easier
to test, I haven't written new unit tests yet.
Thanks,
Josh Elser
