[GitHub] [pulsar] BewareMyPower commented on a change in pull request #12076: [C++] Support setting priority for the consumer

[GitBox <gi...@apache.org>]

BewareMyPower commented on a change in pull request #12076:
URL: https://github.com/apache/pulsar/pull/12076#discussion_r711558463



##########
File path: pulsar-client-cpp/lib/Commands.cc
##########
@@ -258,7 +258,8 @@ SharedBuffer Commands::newSubscribe(const std::string& topic, const std::string&
                                     const std::map<std::string, std::string>& metadata,
                                     const SchemaInfo& schemaInfo,
                                     CommandSubscribe_InitialPosition subscriptionInitialPosition,
-                                    bool replicateSubscriptionState, KeySharedPolicy keySharedPolicy) {
+                                    bool replicateSubscriptionState, KeySharedPolicy keySharedPolicy,
+                                    int priority_level) {

Review comment:
       I suggeste keeping the code style consistent with the existing code. i.e. use low camel case instead of snake case for variables.




-- 
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