abstractdog commented on a change in pull request #117:
URL: https://github.com/apache/tez/pull/117#discussion_r609906867
##########
File path:
tez-runtime-library/src/main/java/org/apache/tez/runtime/library/utils/CodecUtils.java
##########
@@ -78,7 +78,8 @@ public static InputStream
getDecompressedInputStreamWithBufferSize(CompressionCo
throws IOException {
String bufferSizeProp = TezRuntimeUtils.getBufferSizeProperty(codec);
Configurable configurableCodec = (Configurable) codec;
- int originalSize = configurableCodec.getConf().getInt(bufferSizeProp,
DEFAULT_BUFFER_SIZE);
+ int originalSize = bufferSizeProp == null ? DEFAULT_BUFFER_SIZE :
Review comment:
thanks for the fix, I'm about to merge this soon if no objections
@jteagles: could you please contributor rights to @jshmchenxi to the Tez
project in Jira?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
