aloyszhang commented on a change in pull request #8618:
URL: https://github.com/apache/pulsar/pull/8618#discussion_r539097106
##########
File path: pulsar-common/src/main/proto/PulsarApi.proto
##########
@@ -180,6 +180,11 @@ message SingleMessageMetadata {
optional bool null_partition_key = 10 [ default = false];
}
+// raw metadata for message
+message RawMessageMetadata {
Review comment:
> When the broker side gets the message produced from the client, we
could add the property at a new area, which does not combine with
MessageMetadata, and no need deserializing the original MessageMetadata when
gets it out; and when the broker sends the message to the client, we could
choose to filter out this part of the property(or not as the client needs).
As described in
[PIP-70](https://github.com/apache/pulsar/wiki/PIP-70%3A-Introduce-lightweight-raw-Message-metadata),
RawMessageMetadata just avoid serialization and deserialization for the
original MessageMetadata and can be fiter out before send to client.
Here the protobuf message RawMessageMetadata is just a Tool Class for
build/parse Raw meta.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
