jiangpengcheng commented on PR #19470: URL: https://github.com/apache/pulsar/pull/19470#issuecomment-1463456470
> > the type is `String` and available values are: `None`, `LZ4`, `ZLIB`, `ZSTD` and `SNAPPY`, and default to `LZ4` > > @jiangpengcheng thanks for the input. I've drafted [apache/pulsar-site#461](https://github.com/apache/pulsar-site/pull/461) to add the docs. Only one concern - the default config of compression type on client side, e.g., Java producers, is `None` (no compression), which is not aligned with functions. Is there any risk here? There is no risk, just for back compatibility since current pulsar functions are using `LZ4` as compression type. One thing I did wrong is that the `None` should be `NONE` instead -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
