Recursive join rel filter

[andyfengHKU]

No description provided.

[codecov]

Codecov Report

Patch coverage: 100.00% and project coverage change: -0.01 ⚠️

Comparison is base (429d37f) 91.35% compared to head (75358d4) 91.34%.

❗ Current head 75358d4 differs from pull request most recent head 687cfbd. Consider uploading reports for the commit 687cfbd to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1726      +/-   ##
==========================================
- Coverage   91.35%   91.34%   -0.01%     
==========================================
  Files         749      749              
  Lines       27251    27264      +13     
==========================================
+ Hits        24894    24905      +11     
- Misses       2357     2359       +2     

Impacted Files	Coverage Δ
src/include/binder/binder.h	100.00% <ø> (ø)
src/binder/bind/bind_graph_pattern.cpp	95.47% <100.00%> (+0.14%)	⬆️
...inder/bind_expression/bind_subquery_expression.cpp	100.00% <100.00%> (ø)
src/binder/binder.cpp	96.33% <100.00%> (ø)
...c/include/parser/query/graph_pattern/rel_pattern.h	100.00% <100.00%> (ø)
src/parser/transformer.cpp	95.77% <100.00%> (+0.02%)	⬆️

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[ray6080]

Just one general comment, I think it's better to closely follow existing MemGraph syntax, though we didn;t support filter predicates on node variables now, still the syntax should be [e:knows*1..2 (r,n | WHERE list_contains(r.comments, 'rnme'))] or [e:knows*1..2 (r,_ | WHERE list_contains(r.comments, 'rnme'))]