-
Notifications
You must be signed in to change notification settings - Fork 7.1k
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 'initial_query_id' field for system.processors_profile_log #49777
Conversation
Could you add some simple test here? |
This is an automated comment for commit cb130a1 with description of existing statuses. It's updated for the latest CI running
|
FYI: |
This is an automatic comment. The PR descriptions does not match the template. Please, edit it accordingly. The error is: More than one changelog category specified: 'Improvement', '###Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):' |
done. |
@@ -5,16 +5,18 @@ This table contains profiling on processors level (that you can find in [`EXPLAI | |||
Columns: | |||
|
|||
- `event_date` ([Date](../../sql-reference/data-types/date.md)) — The date when the event happened. | |||
- `event_time` ([DateTime64](../../sql-reference/data-types/datetime64.md)) — The date and time when the event happened. |
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.
Thanks for that. event_time is DateTime in the code indeed.
@CheSema |
Here the report.
When you make a tests, it is better to run it locally and just copy .output file as .reference. |
Facilitate profile data association and aggregation for the same query
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Facilitate profile data association and aggregation for the same query