Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Get message by messageId #2912

Closed
codelipenghui opened this issue Nov 2, 2018 · 3 comments · Fixed by #6331
Closed

Get message by messageId #2912

codelipenghui opened this issue Nov 2, 2018 · 3 comments · Fixed by #6331
Labels
area/cli help wanted type/feature The PR added a new feature or issue requested a new feature

Comments

@codelipenghui
Copy link
Contributor

Is it possible to support a rest api to get message by messageId? For now we can peek the head message but can't query message by messageId.
@sijie sijie added type/feature The PR added a new feature or issue requested a new feature area/cli triage/week-45 labels Nov 5, 2018
@foreversunyao
Copy link
Contributor

rest api could have bad performance to do this ?

@codelipenghui
Copy link
Contributor Author

@foreversunyao I'm not sure ! Maybe better read from bookie straightforward.

@sijie sijie mentioned this issue Dec 27, 2019
Closed
@sijie
Copy link
Member

sijie commented Jan 14, 2020

I think reading a message by a message-id is the same as peeking messages.

@nlu90 nlu90 mentioned this issue Feb 15, 2020
Merged
1 task
codelipenghui pushed a commit that referenced this issue Apr 22, 2020
@nlu90
[Issue 2912][pulsar-admin] add get-message-by-id cmd into pulsar-admin (
d3ac61f 
#6331)

Fixes #2912 

### Motivation
Adding a new command `get-message-by-id` to the pulsar-admin which allows user to look at a single message by providing ledger id and entry id.

### Modifications
- pulsar-admin includes the new command `get-message-by-id`
- pulsar-broker v1/v2 apis to handle the get-message-by-id request from pulsar-admin
- managedCursor to read from ledgers with given ledger id and entry id
huangdx0726 pushed a commit to huangdx0726/pulsar that referenced this issue Aug 24, 2020
@nlu90 @huangdx0726
[Issue 2912][pulsar-admin] add get-message-by-id cmd into pulsar-admin (
8b2822a 
apache#6331)

Fixes apache#2912 

### Motivation
Adding a new command `get-message-by-id` to the pulsar-admin which allows user to look at a single message by providing ledger id and entry id.

### Modifications
- pulsar-admin includes the new command `get-message-by-id`
- pulsar-broker v1/v2 apis to handle the get-message-by-id request from pulsar-admin
- managedCursor to read from ledgers with given ledger id and entry id
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/cli help wanted type/feature The PR added a new feature or issue requested a new feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Issue 2912][pulsar-admin] add get-message-by-id cmd into pulsar-admin streamnative/pulsar-archived
3 participants
@sijie @codelipenghui @foreversunyao