anshuksi282-ksolves commented on PR #8601: URL: https://github.com/apache/hadoop/pull/8601#issuecomment-4965933566
Hi @ayushtkn @dineshchitlangia, thank you for reviewing #7009 earlier. I'm new to contributing to Hadoop and have picked this issue up since the original PR went stale. I've addressed the feedback from that PR: - Added a unit test (`TestCodec#testLzoBufferSizeDefault`) verifying the corrected default, as requested by @brahmareddybattula. - Clarified in the description why this restores the pre-existing hadoop-lzo native default (256KB) rather than changing an established one, per @ayushtkn's earlier comment. Would appreciate your review whenever you get a chance. Happy to make any further changes needed. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
