Skip to content

Conversation

@momo-jun
Copy link
Contributor

@momo-jun momo-jun commented Apr 19, 2022

Motivation

To align with code PR #14542 , #14569 , and #13740 .

Modifications

  1. Add managedCursorInfoCompressionType to the config section of broker and standalone.
  2. Update the description of tlsProvider to the config section of broker, standalone, and proxy.
  3. Add tlsProvider to the config section of websocket.
  4. Add webServiceTlsProvider to the config section of broker, standalone, and proxy.

Documentation

  • doc

@github-actions
Copy link

@momo-jun:Thanks for your contribution. For this PR, do we need to update docs?
(The PR template contains info about doc, which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)

@momo-jun
Copy link
Contributor Author

@nodece @codelipenghui can you pls review this doc PR? Thanks.
CC @Anonymitaet.

@github-actions
Copy link

@momo-jun:Thanks for providing doc info!

@github-actions github-actions bot added doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. and removed doc-label-missing labels Apr 19, 2022
Copy link
Member

@nodece nodece left a comment

Choose a reason for hiding this comment

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

LGTM

@momo-jun
Copy link
Contributor Author

momo-jun commented May 5, 2022

Hi @nodece I added docs for #13740. Can you pls review again? Thanks.

@momo-jun
Copy link
Contributor Author

/pulsarbot run-failure-checks

@Anonymitaet Anonymitaet merged commit 5678409 into apache:master May 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Your PR contains doc changes, no matter whether the changes are in markdown or code files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants