Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Accumulo, mail # dev - Review Request 15943: ACCUMULO-1556 - Clarify initialization error for used fs [master]


Copy link to this message
-
Re: Review Request 15943: ACCUMULO-1556 - Clarify initialization error for used fs [master]
Eric Newton 2013-12-02, 18:05

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