asafm commented on issue #16680: URL: https://github.com/apache/pulsar/issues/16680#issuecomment-1193321676
Ok, what you just wrote, if I understand correctly is pretty big and must be well documented in the proposal. You're saying that you will change the way batching works today. If today I have certain knobs which controls how batching is preparing: max batch size, timeout, max number of records and that's it, now you're changing it such that the batching will grouped by the properties defined (`batchedFilterProperties`). This might have serious performance hit not? Might generate smaller batches or increase latency? Why not have list of values for property: region=us,eu ? -- 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]
