-
Notifications
You must be signed in to change notification settings - Fork 264
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
Fixes issue where refresh connections was not running when service was already running. Closes #3734 #3735
Fixes issue where refresh connections was not running when service was already running. Closes #3734 #3735
Conversation
@@ -324,8 +324,8 @@ jobs: | |||
- "blank_aggregators" | |||
- "search_path" | |||
- "chaos_and_query" | |||
- "dynamic_schema" | |||
- "dynamic_aggregators" | |||
# - "dynamic_schema" |
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.
rather than comment them out, please skip the tests with a reason and a link to the issue
# Aggregating two connections with same table and same columns defined. | ||
@test "dynamic aggregator - same table and columns" { | ||
skip "re-design without STEAMPIPE_SYNC_REFRESH" |
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.
comment should read currently does not pass due to bug: <link to issue>
No description provided.