[jira] [Commented] (KAFKA-16397) Use ByteBufferOutputStream to avoid array copy

2024-03-21 Thread Chia-Ping Tsai (Jira)


[ 
https://issues.apache.org/jira/browse/KAFKA-16397?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17829528#comment-17829528
 ] 

Chia-Ping Tsai commented on KAFKA-16397:


[~chiacyu] we all love performance improvement. Let me assign this issue to you

[~apoorvmittal10] it will be great if you can take a look at the PR :)

> Use ByteBufferOutputStream to avoid array copy
> --
>
> Key: KAFKA-16397
> URL: https://issues.apache.org/jira/browse/KAFKA-16397
> Project: Kafka
>  Issue Type: Improvement
>Reporter: Chia-Ping Tsai
>Assignee: Chia-Ping Tsai
>Priority: Minor
>
> from https://github.com/apache/kafka/pull/15148#discussion_r1531889679
> source code:
> https://github.com/apache/kafka/blob/trunk/clients/src/main/java/org/apache/kafka/common/telemetry/internals/ClientTelemetryUtils.java#L216
> we can use ByteBufferOutputStream to collect the uncompressed data, and then 
> return the inner buffer directly instead of copying full array.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (KAFKA-16397) Use ByteBufferOutputStream to avoid array copy

2024-03-21 Thread Apoorv Mittal (Jira)


[ 
https://issues.apache.org/jira/browse/KAFKA-16397?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17829519#comment-17829519
 ] 

Apoorv Mittal commented on KAFKA-16397:
---

[~chia7712] Yes please, go ahead.

> Use ByteBufferOutputStream to avoid array copy
> --
>
> Key: KAFKA-16397
> URL: https://issues.apache.org/jira/browse/KAFKA-16397
> Project: Kafka
>  Issue Type: Improvement
>Reporter: Chia-Ping Tsai
>Priority: Minor
>
> from https://github.com/apache/kafka/pull/15148#discussion_r1531889679
> source code:
> https://github.com/apache/kafka/blob/trunk/clients/src/main/java/org/apache/kafka/common/telemetry/internals/ClientTelemetryUtils.java#L216
> we can use ByteBufferOutputStream to collect the uncompressed data, and then 
> return the inner buffer directly instead of copying full array.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (KAFKA-16397) Use ByteBufferOutputStream to avoid array copy

2024-03-21 Thread Chia Chuan Yu (Jira)


[ 
https://issues.apache.org/jira/browse/KAFKA-16397?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17829420#comment-17829420
 ] 

Chia Chuan Yu commented on KAFKA-16397:
---

Hi, [~apoorvmittal10] 

I'm interested in this enhancement, could I take this one?

Thanks!

> Use ByteBufferOutputStream to avoid array copy
> --
>
> Key: KAFKA-16397
> URL: https://issues.apache.org/jira/browse/KAFKA-16397
> Project: Kafka
>  Issue Type: Improvement
>Reporter: Chia-Ping Tsai
>Priority: Minor
>
> from https://github.com/apache/kafka/pull/15148#discussion_r1531889679
> source code:
> https://github.com/apache/kafka/blob/trunk/clients/src/main/java/org/apache/kafka/common/telemetry/internals/ClientTelemetryUtils.java#L216
> we can use ByteBufferOutputStream to collect the uncompressed data, and then 
> return the inner buffer directly instead of copying full array.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (KAFKA-16397) Use ByteBufferOutputStream to avoid array copy

2024-03-20 Thread Chia-Ping Tsai (Jira)


[ 
https://issues.apache.org/jira/browse/KAFKA-16397?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17829359#comment-17829359
 ] 

Chia-Ping Tsai commented on KAFKA-16397:


[~apoorvmittal10] FYI, and please let me know if you have no free cycle :)

> Use ByteBufferOutputStream to avoid array copy
> --
>
> Key: KAFKA-16397
> URL: https://issues.apache.org/jira/browse/KAFKA-16397
> Project: Kafka
>  Issue Type: Improvement
>Reporter: Chia-Ping Tsai
>Priority: Minor
>
> from https://github.com/apache/kafka/pull/15148#discussion_r1531889679
> source code:
> https://github.com/apache/kafka/blob/trunk/clients/src/main/java/org/apache/kafka/common/telemetry/internals/ClientTelemetryUtils.java#L216
> we can use ByteBufferOutputStream to collect the uncompressed data, and then 
> return the inner buffer directly instead of copying full array.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)