<p></p>
<p><b>@sqrrm</b> requested changes on this pull request.</p>

<hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/4950#discussion_r544221783">core/src/main/java/bisq/core/offer/OpenOfferManager.java</a>:</p>
<pre style='color:#555'>>          }
+
+        OpenOffer openOffer = list.remove(0);
+        if (!openOffers.contains(openOffer) || openOffer.isDeactivated()) {
</pre>
<p>Shouldn't this be <code>!openOffer.isDeactivated()</code> since it's being republished. That was the check before this change.</p>

<hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/4950#discussion_r544227910">core/src/main/java/bisq/core/offer/OpenOfferManager.java</a>:</p>
<pre style='color:#555'>>          }
+
+        OpenOffer openOffer = list.remove(0);
+        if (!openOffers.contains(openOffer) || openOffer.isDeactivated()) {
+            republishOffers(list);
+        }
+
+        republishOffer(openOffer,
+                () -> UserThread.runAfter(() -> republishOffers(list),
+                        30, TimeUnit.MILLISECONDS));
</pre>
<p>I don't understand the reason for the delay, all offers will be prepared to republish recursively, then during the recursion unwind they will all get this call so they delay is likely less than 1ms between each offer republish, but all the calls to republish will happen 30ms later. I think this is not much different to republishing without delay.</p>
<p>The previous version had a random delay, but it was a multiple of the number of orders and 700, so 200 orders ran into 140 seconds republish time and each offer would be spaced out in that time.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/bisq-network/bisq/pull/4950#pullrequestreview-553594831">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNW4ZXYZOTON6VN4IQ3SVCLZZANCNFSM4U3UOSDQ">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AJFFTNTZYJYV6NPQZ4CV3NTSVCLZZA5CNFSM4U3UOSD2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOED7S7TY.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/bisq-network/bisq/pull/4950#pullrequestreview-553594831",
"url": "https://github.com/bisq-network/bisq/pull/4950#pullrequestreview-553594831",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>