Done
Details
Assignee
Alexander KurashAlexander KurashReporter
Julian LadischJulian LadischLabels
Priority
P2Story Points
0Sprint
NoneDevelopment Team
VegaTestRail: Cases
Open TestRail: CasesTestRail: Runs
Open TestRail: Runs
Details
Details
Assignee
Alexander Kurash
Alexander KurashReporter
Julian Ladisch
Julian LadischLabels
Priority
Story Points
0
Sprint
None
Development Team
Vega
TestRail: Cases
Open TestRail: Cases
TestRail: Runs
Open TestRail: Runs
Created January 21, 2022 at 11:06 PM
Updated March 10, 2022 at 2:48 PM
Resolved March 10, 2022 at 2:48 PM
The underlying problem of the bug MODPUBSUB-210:
Future<Boolean>
is used with true indicating success and false indicating failure. This results in two ways how a failure can be propagated: As a failing Future, or as a succeeding Future returning false. This results in bloated code and is error-prone.The idiomatic way is to only use the built-in success/failure indicator of the Future and to use Void as the type:
Future<Void>
.Task: Replace
Future<Boolean>
byFuture<Void>
and use the built-in error handling of Vert.x' Future: https://github.com/folio-org/raml-module-builder/blob/master/doc/futurisation.md#result-and-error-handlingThis is an umbrella to split this task into smaller chunks.