Re: Add Guava as a dependency?

2011-02-11 Thread Luke Lu
+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

Re: Add Guava as a dependency?

2011-02-11 Thread Todd Lipcon
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

Re: Add Guava as a dependency?

2011-02-11 Thread Owen O'Malley
On Feb 11, 2011, at 10:01 AM, Todd Lipcon wrote: 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 :) +1 on

Re: Add Guava as a dependency?

2011-02-11 Thread Arun C Murthy
+1 Arun On Feb 10, 2011, at 7:45 PM, Todd Lipcon 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:

Re: Add Guava as a dependency?

2011-02-11 Thread Mathias Herberts
+1, Guava is the magical piece that can get rid of all those UnsupportedEncodingException ...!

Add Guava as a dependency?

2011-02-10 Thread Todd Lipcon
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:

Re: Add Guava as a dependency?

2011-02-10 Thread Konstantin Boudnik
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

Re: Add Guava as a dependency?

2011-02-10 Thread Jakob Homan
+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: