On Mon, Apr 19, 2021 at 09:44:05PM +0900, Amit Langote wrote:
> Okay, how about the attached then?
create_estate_for_relation() returns an extra resultRelInfo that's
also saved within es_opened_result_relations. Wouldn't is be simpler
to take the first element from es_opened_result_relations instead?
Okay, that's a nit and you are documenting things in a sufficient way,
but that just seemed duplicated to me.
> I decided to go with just
> finish_estate(), because we no longer have to do anything relation
> specific there.
Fine by me.
--
Michael
