On 20.03.26 14:42, Ashutosh Bapat wrote:
>> We should either:
>>
>> (1) Support G041 fully -- both directions. Junwang's patch handles
>> this cleanly with minimal code change, or
>> (2) Not support G041 -- reject all non-local element pattern
>> references, throwing an error for both directions.
>>
>> What do you think?
>
> We will support the cross variable references in future once we
> support more path patterns leading to a more mature code in that area.
> But it doesn't harm to support backward references which can be easily
> supported now. I don't see the code to do so getting disturbed as it
> matures. For the segmentation fault a simple fix to add graph_pattern
> suffices for now. Let's see if Peter has a different opinion.
I think we should not allow non-local references in either direction
(until we implement feature G041), option 2 above. Allowing only
backward references seems kind of an arbitrary implementation artifact.
