marekczajkowski commented on PR #22517: URL: https://github.com/apache/pulsar/pull/22517#issuecomment-2295865788
> > I have added unit tests. As for configuring the compactor there is no new mechanism. It uses existing 'compactionServiceFactoryClassName' property name to switch the factory that creates compactor > > Thanks. The PR itself is ok. > > Configuring the compactor will require some updates in documentation. It would be great if our documention presents this feature, the purpose of it and shows how to enable it. I guess the minimal approach would be to add the configuration key to `broker.conf` and `standalone.conf` and document the configuration key in the comments. Sure. Both configs updated with documenting comments -- 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]
