accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Eric Newton" <eric.new...@gmail.com>
Subject Re: Review Request 15943: ACCUMULO-1556 - Clarify initialization error for used fs [master]
Date Mon, 02 Dec 2013 18:05:00 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15943/#review29603
-----------------------------------------------------------

Ship it!


Ship It!

- Eric Newton


On Dec. 2, 2013, 3:41 p.m., Bill Havanki wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15943/
> -----------------------------------------------------------
> 
> (Updated Dec. 2, 2013, 3:41 p.m.)
> 
> 
> Review request for accumulo and John Vines.
> 
> 
> Bugs: ACCUMULO-1556
>     https://issues.apache.org/jira/browse/ACCUMULO-1556
> 
> 
> Repository: accumulo
> 
> 
> Description
> -------
> 
> Patch for ACCUMULO-1556 for the master branch, as requested by vines. I post this review
because of the shift from FileSystem to VolumeManager, so having extra eyes on it is good
IMO.
> 
> 
> Diffs
> -----
> 
>   server/base/pom.xml c2054bbe3bd7b7db3aa2e209e2a67b9c1c2dc1ad 
>   server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java 0477a443e307e174965140f0fd14c898a0d98727

>   server/base/src/test/java/org/apache/accumulo/server/init/InitializeTest.java PRE-CREATION

> 
> Diff: https://reviews.apache.org/r/15943/diff/
> 
> 
> Testing
> -------
> 
> Unit test added and passed. I had to comment out a test from the earlier branches due
to being unable to mock a static method on FileSystem (and I didn't want to introduce Powermock
just for that).
> 
> 
> Thanks,
> 
> Bill Havanki
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message