Re: [PR] (feat)/sdk client compression [iggy]

["hubcio (via GitHub)" <gi...@apache.org>]

hubcio commented on code in PR #2529:
URL: https://github.com/apache/iggy/pull/2529#discussion_r2653331386


##########
core/sdk/src/clients/consumer.rs:
##########
@@ -980,6 +977,48 @@ impl Stream for IggyConsumer {
                                 message.payload = Bytes::from(payload);
                                 message.header.payload_length = message.payload.len() as u32;
                             }
+
+                            // maybe decompress
+                            if let Ok(Some(algorithm_value)) = message

Review Comment:
   perhaps put this part into `maybe_decompress`? i'm not insisting, i'd like to hear you opinion about 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: commits-unsubscribe@iggy.apache.org

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