[ http://issues.apache.org/jira/browse/HADOOP-538?page=comments#action_12446345 ] Arun C Murthy commented on HADOOP-538: --------------------------------------
IMHO src/native is par for the course since src/c++ contains librecordio and libhdfs (libmapred in future?) which are sortof 'add-ons' to hadoop while HADOOP-538 is 'core-hadoop' in a way. Hence we don't want to mix these up in src/c++. Another reason I wouldn't go down the path of src/c++/libhadoopzip because very soon we might need src/c++/libhadooplzo (for lzo) or something similar... and we don't want to proliferate share-objects needed for hadoop. Given all this and the fact that we can house all of 'native' hadoop code (as and when required) in a single 'src/native' I would go for the current scheme, which can nicely create a single libhadoop.so for all of hadoop's native code. > Implement a nio's 'direct buffer' based wrapper over zlib to improve > performance of java.util.zip.{De|In}flater as a 'custom codec' > ----------------------------------------------------------------------------------------------------------------------------------- > > Key: HADOOP-538 > URL: http://issues.apache.org/jira/browse/HADOOP-538 > Project: Hadoop > Issue Type: Improvement > Affects Versions: 0.6.1 > Reporter: Arun C Murthy > Assigned To: Arun C Murthy > Fix For: 0.8.0 > > Attachments: HADOOP-538.patch, HADOOP-538_20061005.tgz, > HADOOP-538_20061011.tgz, HADOOP-538_20061026.tgz, HADOOP-538_20061030.tgz, > HADOOP-538_benchmarks.tgz > > > There has been more than one instance where java.util.zip's {De|In}flater > classes perform unreliably, a simple wrapper over zlib-1.2.3 (latest stable) > using java.nio.ByteBuffer (i.e. direct buffers) should go a long way in > alleviating these woes. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira