mysql_query: returns execution_time_ms list containing execution time per query

[Andersson007]

SUMMARY

Was requested for c.postgresql and I think it'd be nice to have this here too

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.05%. Comparing base (a45a0d0) to head (e7e5ed8).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #697      +/-   ##
==========================================
- Coverage   79.43%   77.05%   -2.38%     
==========================================
  Files          32       20      -12     
  Lines        2815     2672     -143     
  Branches      703      678      -25     
==========================================
- Hits         2236     2059     -177     
- Misses        397      415      +18     
- Partials      182      198      +16     

Flag	Coverage Δ
integration	76.42% <100.00%> (+0.07%)	⬆️
sanity	?
units	?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Andersson007]

ready for review

[laurent-indermuehle]

Nice addition @Andersson007. I think this is ready.

I'm surprised this feature is not described in DB API 2.0 (PEP 249). Otherwise PyMySQL could have take care of this.

[laurent-indermuehle]

I have just one question about line 159.
Since tests only checks if the returned value is bigger than 0, we never check what's returned.

[Andersson007]

I have just one question about line 159. Since tests only checks if the returned value is bigger than 0, we never check what's returned.

@laurent-indermuehle your suggestion has been committed, also answered the question

[Andersson007]

@markuman @laurent-indermuehle thanks for reviewing!
@optica-phoffmann thanks for requesting the feature!