-
Notifications
You must be signed in to change notification settings - Fork 274
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
Fix issue where service is not shutdown if interrupted while interactive prompt is initialising. Closes #1004 #1280
Conversation
2444a87
to
6d833b2
Compare
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.
looks good - minor tweaks
interactive/interactive_client.go
Outdated
client.Close() | ||
} | ||
|
||
// init data cleanup |
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.
need a better comment here - this is important - this cleanup shuts the service
// cleanup the init data to ensure any services we started are stopped
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.
👍
@@ -7,6 +7,7 @@ import ( | |||
"time" | |||
|
|||
"github.com/turbot/steampipe/db/db_common" | |||
"github.com/turbot/steampipe/workspace" | |||
|
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.
remove gap
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.
done
No description provided.