Query Execution Time is incorrect [APE-773]

[fubuloubu]

Just tracks the time it takes to get an iterator reference, not actually process the data: https://github.com/ApeWorX/ape/blob/806debbfeffd11c816a87c3644b6f8aadf0717f1/src/ape/managers/query.py#L175

to do this correctly, it should do yield from result instead of return result, and then track what the after it's finished yielding

[antazoey]

it willl differ from machine to machine, and internet connection and all sorts of other individual variants.

can we have more info on what exact deliverable requested here?

[fubuloubu]

That the query execution time mentioned din the debug log matches the actually time to execute the query and retrieve all results (currently it only tracks the time to assemble the query results)

Its definitely not critical, but the number rit says currently does not take this into account

[antazoey]

Oh I see, sorry now I understand

[jeamick]

Hello, Is there nyone who picked, it up or is it up for a pull request?

[antazoey]

Hello, Is there nyone who picked, it up or is it up for a pull request?

This one is available to be picked-up.