<p></p>
<p>Notes about commit <a class="commit-link" data-hovercard-type="commit" data-hovercard-url="https://github.com/bisq-network/bisq/commit/2a9d1f6d3460a9c951fe17a39b61153e5723696f/hovercard" href="https://github.com/bisq-network/bisq/commit/2a9d1f6d3460a9c951fe17a39b61153e5723696f"><tt>2a9d1f6</tt></a> (WIP)</p>
<p><a class="user-mention" data-hovercard-type="user" data-hovercard-url="/users/cbeams/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/cbeams">@cbeams</a>,</p>
<p>I know you will look at all of the changes, but please review my use of the Tuple2 return type used in CoreApi?  Wrapping the return value and an ApiStatus code in a single return value -- for void return types too, unfortunately -- seemed like a better idea than throwing Exceptions to be caught by GrpcServer, to then be transformed into gRPC StatusRuntimeExceptions for the client.  I don't know what your preference is.  I prefer to make the changes after you ask for them.</p>
<p>And there is a lot of redundant error handling code in GrpcServer:</p>
<pre><code>if (!result.second.equals(ApiStatus.OK)) {
        StatusRuntimeException ex = new StatusRuntimeException(result.second.getGrpcStatus()
                .withDescription(result.second.getDescription()));
        responseObserver.onError(ex);
        throw ex;
}

</code></pre>
<p>There are only service classes in GrpcServer, no private methods yet.  I don't know if you want these duplicate code blocks factored out.</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/4214#issuecomment-622558564">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNSATXT4LN3ZG5L67S3RPMXRLANCNFSM4MUIOB4Q">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AJFFTNTKUDBWGQAPWZHE6PDRPMXRLA5CNFSM4MUIOB42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEUNX2ZA.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/4214#issuecomment-622558564",
"url": "https://github.com/bisq-network/bisq/pull/4214#issuecomment-622558564",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>