-
Notifications
You must be signed in to change notification settings - Fork 986
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
add new dbt invocation command #6822
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
||
:::tip | ||
|
||
To cancel an active session in the terminal, use the `Ctrl + Z` shortcut. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@vfcoelho i think it'd be helpful to have a 'get out' tip when the session is hanging -- just confirming this is right? it worked for me!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, good thinking! We also have another case that the user's session invocation pool is being occupied by the vscode extension. I wonder if there is any way to cancel its invocations if needed. @dichenqiandbt would you know anything about that so we can add to the tips?
</Expandable> | ||
|
||
<FAQ path="Troubleshooting/long-sessions-cloud-cli" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hiya @mirnawong1
This is a non-blocker but would it be worth making this an expandable or the other expandables, toggles so that they follow the same flow / pattern?
Kind Regards
Natalie
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i added it as a faq so it can be fetched but maybe we can turn the other expandables in detailsToggle so its consistent?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hey @nataliefiann , going to turn the rest into details toggle so they follow the sme ux
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hiya @mirnawong1
Thanks for creating this PR! It was a great read!
I've approved this for you and left one non-blocking suggestion.
Kind Regards
Natalie
|
||
:::tip | ||
|
||
To cancel an active session in the terminal, use the `Ctrl + Z` shortcut. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, good thinking! We also have another case that the user's session invocation pool is being occupied by the vscode extension. I wonder if there is any way to cancel its invocations if needed. @dichenqiandbt would you know anything about that so we can add to the tips?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great doc! Just a few nits I added as comments.
Co-authored-by: vfcoelho <[email protected]>
Co-authored-by: vfcoelho <[email protected]>
Co-authored-by: vfcoelho <[email protected]>
this pr adds a new dbt cloud cli command:
dbt invocation
, which enables users to view active invocations when commands are taking too long or hanging.Resolves #6820
🚀 Deployment available! Here are the direct links to the updated files: