-
Notifications
You must be signed in to change notification settings - Fork 37
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
Multi-line / New Lines in NGQL statements cause syntax errors #220
Comments
As a workaround currently I'm doing some process on our files and passing that into nebula-console, but its def a little hacky and more prone to bugs:
Im using envsubst to replace sensitive variables in the sql |
Now, my way to mitigate this is via @veezhang what do you think, please about this, could we find a way to make it multiple lines aware yet not cause new problems? @bradenwright-opunai also, it's more than welcome to try make the changes(and PR) and see if there are any regressions! Thanks! |
Thanks for the reply, I'll see what veezhang says, depending on the code/complexity that maybe something I can do. I'm working on it as a consultant, so I can spend a little time on my own but if it gets too involved I'll have to get approval from the team I'm doing the work for. |
@bradenwright-opunai The @wey-gu ’s method is very good! Additionally, you can use
|
Hi, I have noticed that the issue you created hasn’t been updated for nearly a month, so I have to close it for now. If you have any new updates, you are welcome to reopen this issue anytime. Thanks a lot for your contribution anyway 😊 这个 issue 超过一个月没有更新内容了,这边先行关闭了。如果你有任何更新的内容,可以再打开这个 issue 哈。 |
Describe the bug (required)
New lines in a NGQL statement seem to break it.
Your Environments (required)
uname -a
:Darwin bradens-MacBook-Pro 22.5.0 Darwin Kernel Version 22.5.0: Mon Apr 24 20:53:19 PDT 2023; root:xnu-8796.121.2~5/RELEASE_ARM64_T6020 arm64
How To Reproduce(required)
Add new lines to a statement and run them, for example this file won't work:
but this will:
Steps to reproduce the behavior:
Expected behavior
I'd expect ; to end a statement and white space to be allowed.
Additional context
It becomes an issue b/c our developers create nebula statements in Nebula Studio and then we automate what they create but the statements in Nebula Studio work with newlines, so it creates errors when we try to automate their statements. Plus we want the ngql statements to be organized / more readable.
The text was updated successfully, but these errors were encountered: