lhotari commented on PR #25805: URL: https://github.com/apache/pulsar/pull/25805#issuecomment-4516671637
Another Claude finding > New error log on pendingReceives.isEmpty() branch can false-fire under cancellation > notifyPendingReceivedCallback is called from ConsumerImpl.java:1383 after hasNextPendingReceive() returns true, but a concurrent cancel()/timeout on the pending future can empty > pendingReceives before this method runs (and the cancel runs on the same internalPinnedExecutor, but not all flows do). Logging this at ERROR with "you encountered a bug" risks alert > fatigue. Consider DEBUG, or also gate by pendingReceives being unexpectedly empty despite no recent cancellation. -- 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]
