ActiveContext reset method

Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.

Problem

The reset method in the ActiveContext indexer concern must remove anything else built on top of @refs.

This could lead to issues if this requirement is missed for new indexer types

Proposal

  • Refactor reset method to raise NotImplementedError and let the consumers handle it
  • Add a shared example that verify @refs is always cleared when reset is called

The following discussion from !178705 (merged) should be addressed:

  • @terrichu started a discussion: (+2 comments)

    question: is this comment something that should be done in the future?

Edited by 🤖 GitLab Bot 🤖