[GitHub] [pulsar] codelipenghui commented on issue #6331: [Issue 2912][pulsar-admin] add get-message-by-id cmd into pulsar-admin

codelipenghui commented on issue #6331: [Issue 2912][pulsar-admin] add get-message-by-id cmd into pulsar-admin URL: https://github.com/apache/pulsar/pull/6331#issuecomment-586851845 @sijie @nlu90 I added `doc-required` tag since this is a new feature for pulsar-admin. It's better to add a

[GitHub] [pulsar] codelipenghui commented on issue #6331: [Issue 2912][pulsar-admin] add get-message-by-id cmd into pulsar-admin

codelipenghui commented on issue #6331: [Issue 2912][pulsar-admin] add get-message-by-id cmd into pulsar-admin URL: https://github.com/apache/pulsar/pull/6331#issuecomment-586704590 @nlu90 Thanks for achieving it. The change looks good to me. I think the `asyncGetMessageById` method

[GitHub] [pulsar] codelipenghui commented on issue #6331: [Issue 2912][pulsar-admin] add get-message-by-id cmd into pulsar-admin

codelipenghui commented on issue #6331: [Issue 2912][pulsar-admin] add get-message-by-id cmd into pulsar-admin URL: https://github.com/apache/pulsar/pull/6331#issuecomment-586546934 @nlu90 I noticed that to get the message content by ID needs to specify a subscription name. Sorry I didn't