Skip to content

enable to specify the sequenceID - #62

Merged
wolfstudy merged 1 commit into
apache:masterfrom
banishee:master
Sep 12, 2019
Merged

enable to specify the sequenceID#62
wolfstudy merged 1 commit into
apache:masterfrom
banishee:master

Conversation

@banishee

Copy link
Copy Markdown
Contributor

Fixes #59

Motivation

When I specify the SequenceID : producer.SendAsync(cox, &pulsar.ProducerMessage{Payload: msg,SequenceID:&id},), it doesn't work.

@wolfstudy
wolfstudy self-requested a review August 26, 2019 11:20
@sijie

sijie commented Sep 4, 2019

Copy link
Copy Markdown
Member

run integration tests

@sijie sijie added this to the 2.5.0 milestone Sep 4, 2019
@wolfstudy
wolfstudy merged commit 11755b2 into apache:master Sep 12, 2019
tisonkun pushed a commit to tisonkun/pulsar-client-go that referenced this pull request Aug 15, 2023
…native/pulsar-admin-go#61)" (apache#62)

This reverts commit c844df17c3857818458e5b90d1b6afb1c5b9a560.
tisonkun pushed a commit to tisonkun/pulsar-client-go that referenced this pull request Aug 15, 2023
tisonkun pushed a commit that referenced this pull request Aug 16, 2023
…native/pulsarctl#61)" (#62)

This reverts commit c844df17c3857818458e5b90d1b6afb1c5b9a560.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

setting the SequenceId can't have an effect on Message deduplication

3 participants