Add new documentation to enable Pulsar PubSub message compression #4990
Conversation
Signed-off-by: Justin Slatten <justin.slatten@gmail.com>
There was a problem hiding this comment.
Pull request overview
This PR adds documentation for new Pulsar message compression features, enabling users to configure compression types and levels for published messages to reduce message size at the cost of CPU usage.
Key Changes:
- Added a new section "Enabling message compression" with compression type and level options
- Documented four compression types:
none,lz4,zlib, andzstd - Documented three compression levels:
default,faster, andbetter
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
daprdocs/content/en/reference/components-reference/supported-pubsub/setup-pulsar.md
Outdated
Show resolved
Hide resolved
| - name: compressionType | ||
| value: lz4 | ||
| - name: compressionLevel | ||
| value: faster | ||
| ``` |
There was a problem hiding this comment.
The new metadata fields compressionType and compressionLevel should be documented in the "Spec metadata fields" table (around line 74-98) for consistency with how other metadata fields are documented in this file. The table should include entries for both fields with their Required status (N), Details (description, valid values, and defaults), and Examples.
There was a problem hiding this comment.
@copilot open a new pull request to apply changes based on this feedback
daprdocs/content/en/reference/components-reference/supported-pubsub/setup-pulsar.md
Show resolved
Hide resolved
…ubsub/setup-pulsar.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Justin Slatten <justin.slatten@gmail.com>
1 participant
Thank you for helping make the Dapr documentation better!
Please follow this checklist before submitting:
In addition, please fill out the following to help reviewers understand this pull request:
Description
Added documentation for new Pulsar message pubsub compressiontype and compressionlevel
Issue reference
#4989