Skip to content

[BP-1.15][FLINK-27399][Connector/Pulsar] [Connector/Pulsar] Change initial consuming position setting logic for better handle the checkpoint.#20564

Merged
tisonkun merged 1 commit into
apache:release-1.15from
streamnative:backport/release-1.15/FLINK-27399
Aug 14, 2022
Merged

[BP-1.15][FLINK-27399][Connector/Pulsar] [Connector/Pulsar] Change initial consuming position setting logic for better handle the checkpoint.#20564
tisonkun merged 1 commit into
apache:release-1.15from
streamnative:backport/release-1.15/FLINK-27399

Conversation

@syhily

@syhily syhily commented Aug 13, 2022

Copy link
Copy Markdown
Contributor

This is a backport PR for #19972.

@flinkbot

flinkbot commented Aug 13, 2022

Copy link
Copy Markdown
Collaborator

CI report:

Bot commands The @flinkbot bot supports the following commands:
  • @flinkbot run azure re-run the last Azure build

…ting logic for better handle the checkpoint. (apache#19972)

* Change the initial start cursor and stop cursor to better handle the consuming behaviors.
* Create the initial subscription instead seek every time. This should fix the wrong position setting.
* Fix the wrong stop cursor, make sure it stops at the correct space
* Drop Consumer.seek() for apache/pulsar#16171
@syhily syhily force-pushed the backport/release-1.15/FLINK-27399 branch from 9baa6e0 to e214044 Compare August 13, 2022 16:52
@tisonkun tisonkun requested a review from MartijnVisser August 14, 2022 02:08
@syhily

syhily commented Aug 14, 2022

Copy link
Copy Markdown
Contributor Author

@PatrickRen Can you help me review this backport issue?

@tisonkun

Copy link
Copy Markdown
Member

Merging...

Anyone can leave a review comment after merge.

@tisonkun tisonkun merged commit 5f842cb into apache:release-1.15 Aug 14, 2022
@syhily syhily deleted the backport/release-1.15/FLINK-27399 branch August 14, 2022 15:01
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.

3 participants