Skip to content

Replace RefCountedFuture with something more robust #102514

Open
@DaveCTurner

Description

@DaveCTurner

In #102498 we introduced RefCountedFuture as an expedient fix to the ref-counting of responses which are temporarily solely retained by a PlainActionFuture, as used by the blocking calls to AbstractClient. This is an ugly hack and likely to cause problems in future (e.g. if a timed .get() times out then we leak the ref). This issue tracks the work to remove this hack and replace it with something more robust, such as the proposal in #102507.

(or we could just stop using these blocking calls altogether, especially in production code where this sort of blocking is just not appropriate)

Metadata

Metadata

Assignees

No one assigned

    Labels

    :Distributed Indexing/DistributedA catch all label for anything in the Distributed Indexing Area. Please avoid if you can.>tech debtTeam:Distributed (Obsolete)Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions