You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/08/03 06:13:58 UTC

[GitHub] [pulsar] codelipenghui commented on a change in pull request #11490: [ManagedLedger] Compress managed ledger info

codelipenghui commented on a change in pull request #11490:
URL: https://github.com/apache/pulsar/pull/11490#discussion_r681463551



##########
File path: managed-ledger/src/main/proto/MLDataFormats.proto
##########
@@ -124,3 +124,8 @@ message ManagedCursorInfo {
     // Store which index in the batch message has been deleted
     repeated BatchedEntryDeletionIndexInfo batchedEntryDeletionIndexInfo = 7;
 }
+
+message ManagedLedgerInfoMetadata {
+    required string compressionType = 1;

Review comment:
       Looks we'd better to copy the enum type to the MLDataFormats.proto because the managed ledger module can be a independent component.




-- 
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@pulsar.apache.org

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