Skip to content

[cpp] make able to set acktimeout to 0 in consumerconfig#7169

Merged
codelipenghui merged 2 commits intoapache:masterfrom
jiazhai:acktimeout_0
Jun 5, 2020
Merged

[cpp] make able to set acktimeout to 0 in consumerconfig#7169
codelipenghui merged 2 commits intoapache:masterfrom
jiazhai:acktimeout_0

Conversation

@jiazhai
Copy link
Member

@jiazhai jiazhai commented Jun 4, 2020

If user already set UnAckedMessagesTimeoutMs in consumer config, It is not able to set it back to 0. This change fix the issue.

@jiazhai jiazhai changed the title make able to set acktimeout to 0 [cpp] make able to set acktimeout to 0 in consumerconfig Jun 4, 2020
@jiazhai jiazhai self-assigned this Jun 4, 2020
@codelipenghui codelipenghui added this to the 2.6.0 milestone Jun 4, 2020
@codelipenghui
Copy link
Contributor

/pulsarbot run-failure-checks

@codelipenghui
Copy link
Contributor

/pulsarbot run-failure-checks

@codelipenghui codelipenghui merged commit a698637 into apache:master Jun 5, 2020
cdbartholomew pushed a commit to kafkaesque-io/pulsar that referenced this pull request Jul 24, 2020
If user already set UnAckedMessagesTimeoutMs in consumer config, It is not able to set it back to 0. This change fix the issue.
huangdx0726 pushed a commit to huangdx0726/pulsar that referenced this pull request Aug 24, 2020
If user already set UnAckedMessagesTimeoutMs in consumer config, It is not able to set it back to 0. This change fix the issue.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants