You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
A new function AR_EXP_ContainsAgg() was created to check if AR_ExpNode tree contains an aggregating function.
If an aggregating function is found, then the function sets the error message and returns true, otherwise, returns false
The function FilterTree_Valid() was modified to receive a second parameter cypher_astnode_type_t type. In case of pattern comprehension or list comprehension type, there is a new validation to check that the predicate is not using aggregation functions.
To validate list comprehension, the function ``_AR_ExpNodeFromComprehensionFunction()``` was modified:
Validate predicate filters using CYPHER_AST_LIST_COMPREHENSION type
Validate that aggregation function is not being used in evaluation node
To validate pattern comprehension, the function buildPatternComprehensionOps() was modified:
Validate that aggregation function is not being used in evaluation node
Validate filters using CYPHER_AST_PATTERN_COMPREHENSION type
The text was updated successfully, but these errors were encountered:
Created by: @nafraf
Source: RedisGraph/RedisGraph#2902
Hi, this PR is to solve #2267. The proposed changes are:
A new function
AR_EXP_ContainsAgg()
was created to check if AR_ExpNode tree contains an aggregating function.If an aggregating function is found, then the function sets the error message and returns true, otherwise, returns false
The function
FilterTree_Valid()
was modified to receive a second parametercypher_astnode_type_t type
. In case of pattern comprehension or list comprehension type, there is a new validation to check that the predicate is not using aggregation functions.To validate list comprehension, the function ``_AR_ExpNodeFromComprehensionFunction()``` was modified:
CYPHER_AST_LIST_COMPREHENSION
typeTo validate pattern comprehension, the function
buildPatternComprehensionOps()
was modified:CYPHER_AST_PATTERN_COMPREHENSION
typeThe text was updated successfully, but these errors were encountered: