<p><a class="user-mention" data-hovercard-type="user" data-hovercard-url="/users/stejbac/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/stejbac">@stejbac</a> Thanks for the implementation of the suggested changes!</p>
<p>Could we make a list of all potential persisted objects (some are only created in rare cases like FailedTrades) to check for each persisted object if the adequate interface is applied? Would be also great to see a performance comparison before/after change (hope I don't get too annoying with those requests but I think its a bit risky change we better to be extra careful here).</p>
<p>I just found another issue (no serious problem but risk for future problems):<br>
<code>VoteWithProposalTxIdList</code> is extending <code>PersistableList</code> which is now using the <code>UserThreadMappedPersistableEnvelope</code>. This list is using the PersistableList only for serialisation but not real persistence (see comment in class). This should be fixed anyway by a more correct style (e.g. using a dedicated List only for that purpose). As we don't call the <code>toPersistableMessage</code> method here no behaviour change is introduced, but if anyone would start further changes we might inherit a potential critical change, which can easily lead to a consensus problem. I suggest to clean that up either with the PR (or better with a new one) so that this risk by abusing the <code>PersistableList</code> here is removed.</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/4025?email_source=notifications&email_token=AJFFTNST5QAZHUIMB26BOPLRG2EX7A5CNFSM4LBEPX62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOMWXHY#issuecomment-597257119">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNRATR3WCNTFVJ2TZXLRG2EX7ANCNFSM4LBEPX6Q">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AJFFTNXKN3VYNWA7N7GDG6TRG2EX7A5CNFSM4LBEPX62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOMWXHY.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/4025?email_source=notifications\u0026email_token=AJFFTNST5QAZHUIMB26BOPLRG2EX7A5CNFSM4LBEPX62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOMWXHY#issuecomment-597257119",
"url": "https://github.com/bisq-network/bisq/pull/4025?email_source=notifications\u0026email_token=AJFFTNST5QAZHUIMB26BOPLRG2EX7A5CNFSM4LBEPX62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOMWXHY#issuecomment-597257119",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>