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
Log async insert flush queries into to system.query_log and system.processes #51160
Log async insert flush queries into to system.query_log and system.processes #51160
Conversation
This is an automated comment for commit 09a6472 with description of existing statuses. It's updated for the latest CI running
|
There are some issues in the tests, related to the handling of the new |
5406dbc
to
b1cedf0
Compare
I'll keep an eye on the rest of the tests as they advance, but I think this is ok to review now. |
Failed tests are all unrelated:
|
PR is clean and ready for review. Updating with master for the sake of keeping it close to HEAD |
Failure in unrelated test -> #52059 |
"are not logged" -> are now logged. |
I've asked @CurtizJ to review. |
This is a nice add @Algunenano thanks for this!. Am I right into assuming this cannot be backported to LTS 23.3? because until this PR it was difficult to have some introspection on async_inserts apart from the
|
Impossible is nothing, but yes, features aren't backported.
This one is a bug but it should be easily fixable. I'll send the fix. |
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Documentation entry for user-facing changes
Flush queries for async inserts (the queries that do the final push of data) are now logged in the system.query_log where they appear as
query_kind = 'AsyncInsertFlush'
.These flush queries also now appear in system.processes and can be killed as any normal insert, but they don't count towards the limits for max concurrent queries.
Closes #50720