[ 
https://issues.apache.org/jira/browse/HADOOP-17887?focusedWorklogId=646152&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-646152
 ]

ASF GitHub Bot logged work on HADOOP-17887:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 03/Sep/21 02:37
            Start Date: 03/Sep/21 02:37
    Worklog Time Spent: 10m 
      Work Description: viirya commented on a change in pull request #3377:
URL: https://github.com/apache/hadoop/pull/3377#discussion_r701537681



##########
File path: 
hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/zlib/BuiltInGzipCompressor.java
##########
@@ -86,10 +86,6 @@ public int compress(byte[] b, int off, int len) throws 
IOException {
 
     int compressedBytesWritten = 0;
 
-    if (currentBufLen <= 0) {
-      return compressedBytesWritten;
-    }
-

Review comment:
       Do we still need another test for it? We currently have test coverage 
for it.




-- 
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.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Issue Time Tracking
-------------------

    Worklog Id:     (was: 646152)
    Time Spent: 1h 20m  (was: 1h 10m)

> Remove GzipOutputStream
> -----------------------
>
>                 Key: HADOOP-17887
>                 URL: https://issues.apache.org/jira/browse/HADOOP-17887
>             Project: Hadoop Common
>          Issue Type: Improvement
>            Reporter: L. C. Hsieh
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 1h 20m
>  Remaining Estimate: 0h
>
> As we provide built-in gzip compressor, we can use it in compressor stream. 
> The wrapper GzipOutputStream can be removed now.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

Reply via email to