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.
