Cool, seems like enough people are on board. I'll just include this in the patch for HADOOP-7132 (naming the IPC Reader Threads) since that's where I wanted to use it.
Can't wait to use this stuff in more patches. I love guava :) -Todd On Fri, Feb 11, 2011 at 4:09 AM, Luke Lu <l...@vicaya.com> wrote: > +1. guava is the new apache commons, maintained by java experts with > comprehensive test coverage. > > I also propose to deprecate any existing utils in hadoop-common with > duplicate functionality. > > On Thu, Feb 10, 2011 at 10:25 PM, Jakob Homan <jgho...@gmail.com> wrote: > > +1 > > > > On Thu, Feb 10, 2011 at 10:04 PM, Konstantin Boudnik <c...@apache.org> > wrote: > >> Actually it seems that Pig uses this already, which perhaps means it > >> is good enough for us as well ;) > >> -- > >> Take care, > >> Konstantin (Cos) Boudnik > >> > >> On Thu, Feb 10, 2011 at 19:45, Todd Lipcon <t...@cloudera.com> wrote: > >>> Anyone mind if I pull in the Guava library as a dependency for Common? > It > >>> has a bunch of very useful utilities - in this particular case the one > I'm > >>> itching to use is ThreadFactoryBuilder: > >>> > >>> > http://guava-libraries.googlecode.com/svn/tags/release05/javadoc/com/google/common/util/concurrent/ThreadFactoryBuilder.html > >>> > >>> More info here: > >>> http://code.google.com/p/guava-libraries/ > >>> > >>> -Todd > >>> -- > >>> Todd Lipcon > >>> Software Engineer, Cloudera > >>> > >> > > > -- Todd Lipcon Software Engineer, Cloudera