At the moment when querying the database expecting multiple results, we don't care about the status returned. That might lead to incorrect behavior.
Feature
Use DoobieMultipleResultFunctionWithAggStatus instead of DoobieMultipleResultFunction to handle the status returned from the database functions.
Proposed Solution
Solution Ideas:
upgrade to Fa-DB v0.4.0
use the ByFirstErrorStatusAggregator mix-in trait
NB! The import path has changed from za.co.absa.fadb to za.co.absa.db.fadb. Similarly the library path has changed from za.co.absa.fa-db to za.co.absa.db.fa-db. (Mass replace could quickly solve it.)
Background
At the moment when querying the database expecting multiple results, we don't care about the
status
returned. That might lead to incorrect behavior.Feature
Use
DoobieMultipleResultFunctionWithAggStatus
instead ofDoobieMultipleResultFunction
to handle thestatus
returned from the database functions.Proposed Solution
Solution Ideas:
ByFirstErrorStatusAggregator
mix-in traitza.co.absa.fadb
toza.co.absa.db.fadb
. Similarly the library path has changed fromza.co.absa.fa-db
toza.co.absa.db.fa-db
. (Mass replace could quickly solve it.)