[GitHub] [kafka] showuon commented on a diff in pull request #13135: KAFKA-14633: Reduce data copy & buffer allocation during decompression

2023-05-27 Thread via GitHub
showuon commented on code in PR #13135: URL: https://github.com/apache/kafka/pull/13135#discussion_r1207907884 ## clients/src/main/java/org/apache/kafka/common/record/DefaultRecord.java: ## @@ -356,164 +346,100 @@ private static DefaultRecord readFrom(ByteBuffer buffer,

[GitHub] [kafka] showuon commented on a diff in pull request #13135: KAFKA-14633: Reduce data copy & buffer allocation during decompression

2023-05-08 Thread via GitHub
showuon commented on code in PR #13135: URL: https://github.com/apache/kafka/pull/13135#discussion_r1187071361 ## clients/src/main/java/org/apache/kafka/common/utils/ChunkedBytesStream.java: ## @@ -0,0 +1,357 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

[GitHub] [kafka] showuon commented on a diff in pull request #13135: KAFKA-14633: Reduce data copy & buffer allocation during decompression

2023-05-07 Thread via GitHub
showuon commented on code in PR #13135: URL: https://github.com/apache/kafka/pull/13135#discussion_r1185891570 ## clients/src/main/java/org/apache/kafka/common/utils/ChunkedBytesStream.java: ## @@ -0,0 +1,357 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one