tree 2d591afbaeeae652f148c98efee7a7d530768778
parent 44a2a290b7b084b3dfd5aad3617ad4885d56f1ae
author Antonio García-Domínguez <agarcdomi@gmail.com> 1593611851 +0100
committer Antonio García-Domínguez <agarcdomi@gmail.com> 1593611856 +0100

Use overridable factory method for EOLQueryEngine graph node wrapping

GraphNodeWrapper allows us to hold weak references to nodes, to save memory.
However, when using time-aware queries, we cannot just recover the original
node from its ID: we need the timepoint as well. That means that we need to
wrap the node with a different class.

Until now, we basically had it done on a case-by-case basis, but that was
prone to slips: for instance, following a model element reference from a
time-aware node would give you a non-time-aware wrapper. This was OK until
you tried to run a large query, which resulted in GC'ing of the weak references:
from that point onwards, you'd start to see odd NullPointerExceptions in places.