{"push_id":2314100232,"size":1,"distinct_size":1,"ref":"refs/heads/master","head":"84ca73ed135b359b4fd22fa3e501492f94880688","before":"18b209040cd1c0e2ff06a95b3fd1f271d36b925b","commits":[{"sha":"84ca73ed135b359b4fd22fa3e501492f94880688","author":{"name":"Mark","email":"89e2e4a5926da24fe8e8be6d2a9ed5ca4e2f51a7@gmail.com"},"message":"Improve rabbit template\n\n* Remove double synchronized\r\n\r\n* Polish `receiveForReply`\r\n\r\nChange `receiveMessage` from method argument to local variable since it's always passed as null\r\n\r\n* Remove exception casting in `consumerDelivery`\r\n\r\nSince `doExecute()` eventually will still wrap all exceptions using `RabbitExceptionTranslator.convertRabbitAccessException(e)`\r\n\r\nAs now exceptions are being re-thrown move consumer cancelling logic inside finally block\r\n\r\n* Guard `setReplyAddress()`\r\n\r\nUse same lock as `evaluateFastReplyTo()` is guarded via `RabbitTemplate` lock in `doSendAndReceive()`\r\n\r\n* Remove locking on `publisherConfirmChannels` map\r\n\r\nWhen getting unconfirmed messages count/correlation data remove locking on entire map\r\nsince concurrent map iterator's are thread safe and underlying Channel's implementation (`PublisherCallbackChannelImpl`) already use\r\nsynchronization in expire and `getPendingConfirmsCount()`\r\n\r\n* Extract common consumer canceling logic\r\n\r\n* Access `directReplyToContainers` map only via lock\r\n\r\n* remove double checking for null container\r\n\r\n* change copyright\r\n\r\n* Polishing - PR comments\r\n\r\n* Revert \"Access directReplyToContainers map only via lock\"\r\n\r\nThis reverts commit 489df24","distinct":true,"url":"https://api.github.com/repos/spring-projects/spring-amqp/commits/84ca73ed135b359b4fd22fa3e501492f94880688"}]}
{"actor":{"display_login":"artembilan"}}