Permalink
Browse files

Merge pull request #63 from ekoontz/add-buffer-warning

fix typo in prev commit.
  • Loading branch information...
2 parents 0c07467 + 7ce9940 commit 9a6528b91e9a2da20d30fb78941fbbbf54d4278b @rangadi rangadi committed Feb 26, 2013
Showing with 1 addition and 1 deletion.
  1. +1 −1 src/java/com/hadoop/compression/lzo/LzoDecompressor.java
@@ -207,7 +207,7 @@ public synchronized void setInput(byte[] b, int off, int len) {
len + " is greater than this decompressor's directBufferSize: " +
directBufferSize + ". To fix this, increase the value of your " +
"configuration's io.compression.codec.lzo.buffersize to be larger " +
- "than: " + len + ".")
+ "than: " + len + ".");
}
}

4 comments on commit 9a6528b

I could not find how to communicate with you. I ran ant clean test at hadoop-lzo directory and see following error.
[exec] ompressor.c:273:3: warning: implicit declaration of function ‘strstr’ [-Wimplicit-function-declaration]
[exec] /home/deepakkv/softwares/hadoop-lzo/src/native/impl/lzo/LzoCompressor.c:273:14: warning: incompatible implicit declaration of built-in function ‘strstr’ [enabled by default]
[exec] make: *** [impl/lzo/LzoCompressor.lo] Error 1

To fix above i included string.h in /home/deepakkv/softwares/hadoop-lzo/src/native/impl/lzo/LzoCompressor.c and ran into below error
[exec] /home/deepakkv/softwares/hadoop-lzo/src/native/impl/lzo/LzoCompressor.c: In function ‘Java_com_hadoop_compression_lzo_Lzodepbase=echo impl/lzo/LzoCompressor.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||';
[exec] /bin/bash ./libtool --tag=CC --mode=compile gcCompressor_initIDs’:
[exec] /home/deepakkv/softwares/hadoop-lzo/src/native/impl/lzo/LzoCompressor.c:125:37: error: expected expression before ‘,’ token
[exec] make: *** [impl/lzo/LzoCompressor.lo] Error 1

Environment
LIBRARY_PATH=/usr/local/lzo-2.06/lib
JAVA_HOME=/usr/lib/jvm/java-6-openjdk-amd64/
Linux ubuntu 3.2.0-29-generic #46-Ubuntu SMP Fri Jul 27 17:03:23 UTC 2012 x86_64 x86_64 x86_64 GNU/Linux
C_INCLUDE_PATH=/usr/local/lzo-2.06/include

Contributor

rangadi replied Apr 29, 2013

I am not sure how to assign it to you. #65

Contributor

rangadi replied Apr 29, 2013

Please sign in to comment.