Skip to content
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

Report bytes written in query results statistics #18651

Closed
jkylling opened this issue Aug 11, 2023 · 1 comment · Fixed by #18658
Closed

Report bytes written in query results statistics #18651

jkylling opened this issue Aug 11, 2023 · 1 comment · Fixed by #18658
Labels
enhancement New feature or request

Comments

@jkylling
Copy link
Contributor

It would be useful for clients of the HTTP REST API which are iterating over the nextUris of some write operation to know the approximate number of bytes written so far, in addition to the number of bytes processed or CPU time used. This can for instance be used to cancel queries writing too much data early.

The physicalWrittenDataSize is already part of the infoUri, but this URL is more expensive.

The StatementStats created here could be extended to include queryStats.getPhysicalWrittenDataSize().toBytes().

@jkylling jkylling added the enhancement New feature or request label Aug 11, 2023
@electrum
Copy link
Member

This seems reasonable. Feel free to submit a PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

Successfully merging a pull request may close this issue.

2 participants