jrgemignani commented on code in PR #1018:
URL: https://github.com/apache/age/pull/1018#discussion_r1260331429
##########
src/backend/parser/cypher_gram.y:
##########
@@ -2415,7 +2434,7 @@ static cypher_relationship *build_VLE_relation(List
*left_arg,
(cnl->name == NULL && cnr->label != NULL) ||
(cnl->name == NULL && cnr->props != NULL))
{
- cnl->name = create_unique_name("_vle_function_start_var");
+ cnl->name =
create_unique_name(AGE_DEFAULT_PREFIX"vle_function_start_var");
Review Comment:
AGE_DEFAULT_PREFIX does end in an _ -
```
/*
* Every internal alias or variable name should be prefixed
* with AGE_DEFAULT_PREFIX. Grammer restricts variables
* prefixed with _age_default_ in user query to be used.
*/
#define AGE_DEFAULT_PREFIX "_age_default_"
#define AGE_DEFAULT_ALIAS_PREFIX AGE_DEFAULT_PREFIX"alias_"
#define AGE_DEFAULT_VARNAME_PREFIX AGE_DEFAULT_PREFIX"varname_"
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
