RobertIndie commented on code in PR #257:
URL: https://github.com/apache/pulsar-client-cpp/pull/257#discussion_r1177607224
##########
include/pulsar/Message.h:
##########
@@ -176,8 +176,16 @@ class PULSAR_PUBLIC Message {
*/
bool hasSchemaVersion() const;
+ /**
+ * Get the schema version.
+ *
+ * @return the the schema version on success or -1 if the message does not
have the schema version
+ */
+ int64_t getLongSchemaVersion() const;
+
/**
* Get the schema version
+ * @deprecated Use getLongSchemaVersion instead
Review Comment:
It seems that we can also do it on the Java client. If we want to replace
this public API, it's better to propose a PIP and change the Java client side
first. I think we need to keep the API as consistent as possible with the Java
client.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
