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.

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

[Kinesis] Fix kinesis sink connector does not ack messages #10769

Merged
merged 1 commit into from
Jun 2, 2021

Conversation

RobertIndie
Copy link
Member

Motivation

Currently, when the kinesis sink connector sends the message successfully, it will not ack the message.

Modifications

  • Ack messages after the sink connector send messages successfully.

Copy link
Contributor

@fantapsody fantapsody left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@eolivelli
Copy link
Contributor

@dlg99 FYI

@tuteng tuteng added area/connector type/bug The PR fixed a bug or issue reported a bug labels Jun 1, 2021
@tuteng tuteng modified the milestones: 2.9.0, 2.8.0 Jun 1, 2021
@dlg99
Copy link
Contributor

dlg99 commented Jun 1, 2021

@tuteng shouldn't it also do this.resultContext.fail() in the onFailure() case to nack the message?

Copy link
Contributor

@dlg99 dlg99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also it is missing nack in the onFailure() case

@sijie
Copy link
Member

sijie commented Jun 1, 2021

@dlg99 there is a retry logic in the onFailure part.

@codelipenghui codelipenghui merged commit b458b4a into apache:master Jun 2, 2021
wangjialing218 pushed a commit to wangjialing218/pulsar that referenced this pull request Jun 2, 2021
…0769)

### Motivation

Currently, when the kinesis sink connector sends the message successfully, it will not ack the message.

### Modifications

* Ack messages after the sink connector send messages successfully.
zymap pushed a commit that referenced this pull request Jun 2, 2021
### Motivation

Currently, when the kinesis sink connector sends the message successfully, it will not ack the message.

### Modifications

* Ack messages after the sink connector send messages successfully.

(cherry picked from commit b458b4a)
@zymap zymap added the cherry-picked/branch-2.7 Archived: 2.7 is end of life label Jun 2, 2021
yangl pushed a commit to yangl/pulsar that referenced this pull request Jun 23, 2021
…0769)

### Motivation

Currently, when the kinesis sink connector sends the message successfully, it will not ack the message.

### Modifications

* Ack messages after the sink connector send messages successfully.
bharanic-dev pushed a commit to bharanic-dev/pulsar that referenced this pull request Mar 18, 2022
…0769)

### Motivation

Currently, when the kinesis sink connector sends the message successfully, it will not ack the message.

### Modifications

* Ack messages after the sink connector send messages successfully.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/connector cherry-picked/branch-2.7 Archived: 2.7 is end of life release/2.7.3 type/bug The PR fixed a bug or issue reported a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants