lhotari commented on PR #24064: URL: https://github.com/apache/pulsar/pull/24064#issuecomment-2705831384
> Early detection hasBrokerBelowLowerBound This would be a better name for the PR. The "optimization" itself is more about code cleanup than an actual optimization that has an impact. The title should also be specific about the area where this change is made, mentioning the ThresholdShedder. btw. For larger PRs, it's useful to use LLMs like DeepSeek to provide a draft of the PR title and description. Here's a guide of how to achieve that: https://gist.github.com/lhotari/81f533af4b9ad515e02d96e543c4408b . In this case, it wouldn't necessarily be useful, but I highly recommend using LLMs for fixing grammar and making PR titles more specific. Obviously they could make mistakes, but it usually gives a good suggestion. With DeepSeek, there's a challenge since very large PRs will hit the context size limits. With Claude AI Professional, I haven't had that problem. -- 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]
