Go ahead and move on to task 2.  I don't want to be a blocker for you. 

I'll try to check your code out sometime this weekend and provide feedback. 

On Fri, Jul 10, 2015 at 12:10 PM, Rukshan Chathuranga <rcrukshan17@xxxxxxxxx> wrote:

Hi Whitney,

I implemented Test cases for Table[1] and Tablet[2] stats. Also i implement rest services[3][4] for them as well. Can you review it and let me know whether can i moved to task 2.

[1] https://github.com/ruks/geowave/blob/GEOWAVE-Monitor-GSOC15/services/webapp/src/test/java/mil/nga/giat/geowave/service/healthimpl/TableStatsTest.java

[2] https://github.com/ruks/geowave/blob/GEOWAVE-Monitor-GSOC15/services/webapp/src/test/java/mil/nga/giat/geowave/service/healthimpl/TabletStatsTest.java

[3] https://github.com/ruks/geowave/blob/GEOWAVE-Monitor-GSOC15/services/webapp/src/main/java/mil/nga/giat/geowave/service/healthimpl/TableStats.java

[4] https://github.com/ruks/geowave/blob/GEOWAVE-Monitor-GSOC15/services/webapp/src/main/java/mil/nga/giat/geowave/service/healthimpl/TabletStats.java

Thanks and Regards.

Rukshan Chathuranga.

Department Of Computer Science & Engineering,

Faculty Of Engineering,

University Of Moratuwa. Sri Lanka.

WEB: http://www.rukspot.com/

On Wed, Jul 8, 2015 at 8:55 PM, Whitney O'Meara <james.w.omeara@xxxxxxxxx> wrote:

Look into MiniAccumuloCluster.  You can (and should) use that to setup an accumulo cluster with geowave for unit tests. 

On Wed, Jul 8, 2015 at 11:11 AM, Rukshan Chathuranga <rcrukshan17@xxxxxxxxx> wrote:

Hi Whitney,

i implement two test case for each Tablet[1] and Table[2] Stat classed. Can you review and let me know, what should i improve?

Since Accumulo Statistics are changed with time, it is difficult to define constant values to assert them. Also is there any way to automatically start Geowave with test cases. I used the GeoWaveDemoApp.java, but it seems it does not start as expected.

[1] https://github.com/ruks/geowave/blob/GEOWAVE-Monitor-GSOC15/services/webapp/src/test/java/mil/nga/giat/geowave/service/healthimpl/TabletStatsTest.java

[2] https://github.com/ruks/geowave/blob/GEOWAVE-Monitor-GSOC15/services/webapp/src/test/java/mil/nga/giat/geowave/service/healthimpl/TableStatsTest.java

Thanks and Regards.