Field naming starting with reserved keywords breaks updates
See original GitHub issueWhen in a table there is a column that starts with a reserved keyword, for example nullSomeColumnName or trueSomeColumnName, if you try to create an update with an on_conflict and columns, the update will fail like:
extensions": {
"path": "$.selectionSet.insert_some_table.args.on_conflict.update_columns[indexOfColumnThatStartsWithNull]",
"code": "validation-failed"
},
"message": "null value found for non-nullable type: some_table_update_column!"
If the field starts with true or false the error is almost the same but the message is related to value must be an enum.
It seems like the engine is trying to parse the column names somehow.
Issue Analytics
- State:
- Created 4 years ago
- Reactions:1
- Comments:13 (12 by maintainers)
Top Results From Across the Web
Reserved Keywords (Transact-SQL) - Microsoft Learn
Transact-SQL reserved keywords can be used as identifiers or names of databases or database objects, such as tables, columns, views, and so on....
Read more >Reserved Keywords - Real Python
The Python language reserves a small set of keywords that designate special language functionality. No object can have the same name as a ......
Read more >How can this code use reserved keywords as field names?
My IDE decompiled some third party libraries and I'm curious how this can be valid - never saw before keywords can be used...
Read more >Problem names and reserved words in Access - Allen Browne
Num Bad Name Reason identified
1 _Evaluate Form method
2 _Name Form method; Form Section method
3 A_ADD Access constant (old)
Read more >Reserved Keywords in Formulas - Salesforce Help
In CRM Analytics, Data Prep has reserved keywords that need special handling when manually writing formulas. Use double quotes " when referencing dataset......
Read more >
Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free
Top Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Parser changes merged. Referencing commit in engine:
@rstpv I’ve resolved the issue in the parser and am now testing integration with graphql-engine. Should have a candidate fix ready shortly.