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

[GitBox]

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


   > But i am afraid your present implementation will compile failed due to 
lack of `MessageIdImpl` constructor for `@QueryParam("messageId") 
@DefaultValue("latest") MessageIdImpl messageId,`
   
   Yes i was wrong, maybe we should proceed in the following way:
   
   ```
               @ApiParam(name = "messageId", value = "messageId where to create 
the subscription. " +
                       "It can be 'latest', 'earliest' or (ledgerId:entryId)",
                       defaultValue = "latest",
                       allowableValues = "latest,earliest,ledgerId:entryId"
               ) 
               MessageIdImpl messageId,
   ```


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