albertogpz commented on a change in pull request #6052:
URL: https://github.com/apache/geode/pull/6052#discussion_r589348036
##########
File path:
geode-core/src/main/java/org/apache/geode/cache/wan/GatewaySender.java
##########
@@ -174,7 +174,32 @@
*/
int GET_TRANSACTION_EVENTS_FROM_QUEUE_RETRIES =
Integer.getInteger(GeodeGlossary.GEMFIRE_PREFIX +
"get-transaction-events-from-queue-retries",
- 10);
+ 2);
Review comment:
Prior to this change, retries were done immediately if it was detected
that not all events for a transaction were in the batch. In this change, I have
added a configurable sleep before retrying with a default value of 1ms because
I saw that retries were sometimes done so fast that it was not enough to get
the expected events in the queue and it was more reasonable to have less
retries and have some waiting before retrying.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
