-
Notifications
You must be signed in to change notification settings - Fork 406
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Should there be a proper error message when using RETURN * without any variable in scope ? #1030
Comments
Displaying proper error messages indicating a specific problem is helpful in debugging the code and solving the issue easily and efficiently. Since it enhances the user experience a proper error message indicating the main reason why it occurred should be displayed for the said command. |
I suppose the complete query that you have run and has caused the error is as below.
The error is not because of the RETURN clause. It is because of the mismatch of number of columns returned by the cypher query Basically in age there is a function by the name of The following query also cause the same error:
Error messages are always generalized. |
There are other places where error messages are not proper. For example, when age_load tries loading vertices into a non-existing label, it throws 'label id must between x and y,' instead of saying 'label does not exist'. I think a project can be created to rewrite some of these error messages from end users point of view. |
It's a good idea to rewrite some error messages in AGE. I have also encountered issues while coding and debugging When I tried to use the function that I developed |
The error: "Return row and column definition list do not match." occurs when the number of values returned does not match the expected output defined by the RETURN clause. Hence it's appropriate for that query. However, there are other cases in AGE where generating specific errors would help users easily understand the cause of errors. Hence, I think there should be a project for that. |
I think that we should attempt to fix, or align, our messages with, say, Neo4j, where it is necessary or makes sense. |
@MuhammadTahaNaveed Is this still an issue or can this be closed? |
This issue is stale because it has been open 60 days with no activity. Remove "Abondoned" label or comment or this will be closed in 14 days. |
For the question above, neo4j returns the following error
But age generates
Should we also generate a proper error message in transformation stage catching this invalid use ?
The text was updated successfully, but these errors were encountered: