Skip to content

[pulsar-client] Make it possible to disable poolMessages - #12108

Merged
lhotari merged 1 commit into
apache:masterfrom
yuruguo:Disable-pool-message-in-pulsar-client
Sep 24, 2021
Merged

[pulsar-client] Make it possible to disable poolMessages#12108
lhotari merged 1 commit into
apache:masterfrom
yuruguo:Disable-pool-message-in-pulsar-client

Conversation

@yuruguo

@yuruguo yuruguo commented Sep 21, 2021

Copy link
Copy Markdown
Contributor

Motivation

Details are similar to [pulsar-perf] Make it possible to disable poolMessages

Documentation

  • no-need-doc

@Anonymitaet Anonymitaet added the doc-not-needed Your PR changes do not impact docs label Sep 22, 2021
@315157973
315157973 requested a review from lhotari September 22, 2021 06:33
private String schematype = "bytes";

@Parameter(names = { "-pm", "--pool-messages" }, description = "Use the pooled message")
@Parameter(names = { "-pm", "--pool-messages" }, description = "Use the pooled message", arity = 1)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

can we also update the doc here?

@eolivelli eolivelli left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

+1

@lhotari lhotari added this to the 2.9.0 milestone Sep 24, 2021
@lhotari
lhotari merged commit 4aae519 into apache:master Sep 24, 2021
codelipenghui pushed a commit that referenced this pull request Sep 29, 2021
@codelipenghui codelipenghui added the cherry-picked/branch-2.8 Archived: 2.8 is end of life label Sep 29, 2021
bharanic-dev pushed a commit to bharanic-dev/pulsar that referenced this pull request Mar 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/cli cherry-picked/branch-2.8 Archived: 2.8 is end of life doc-not-needed Your PR changes do not impact docs release/2.8.2 release/2.9.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants