[GitHub] [pulsar] hangc0276 commented on pull request #6776: [Docs] Fix the Create subscribtion swagger of PersistentTopic

[GitBox]

hangc0276 commented on pull request #6776:
URL: https://github.com/apache/pulsar/pull/6776#issuecomment-618784699


   > @hangc0276 Good catch! @wolfstudy could please help take a look the last 
comment? maybe you need to move the `messageId` to the end of params list.
   
   Thanks for your feedback. @wolfstudy  maybe you should implement a 
constructor for `MessageIdImpl`，which param is a string, and you should parse 
string to `MessageIdImpl` field, such as ledgerId, entryId and partitionIndex.  
Because the QueryParam just pass a string as param to messgeId, you should 
construct a MessageIdImpl object using the string param.


----------------------------------------------------------------
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:
us...@infra.apache.org