remove execution plan cost extraction #9631

djova · 2021-06-30T21:55:56Z

Extraction of cost from execution plans is now being done during ingestion.

Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
PR title must be written as a CHANGELOG entry (see why)
Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
PR must have changelog/ and integration/ labels attached

Extraction of cost from execution plans is now being done during ingestion.

codecov · 2021-07-08T22:31:10Z

Codecov Report

Merging #9631 (faf33b0) into master (5584fc3) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Flag	Coverage Δ
mysql	`84.84% <100.00%> (+0.06%)`	⬆️

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

djova requested review from a team as code owners June 30, 2021 21:55

ghost added the integration/mysql label Jun 30, 2021

djova added the changelog/Changed label Jun 30, 2021

djova changed the title ~~remove explain plan cost extraction~~ remove execution plan cost extraction Jun 30, 2021

djova force-pushed the djova/mysql-remove-plan-cost branch from c18d9e6 to abd74ed Compare June 30, 2021 21:58

alexbarksdale previously approved these changes Jun 30, 2021

View reviewed changes

remove execution plan cost extraction

980de7e

Extraction of cost from execution plans is now being done during ingestion.

djova dismissed alexbarksdale’s stale review via 980de7e July 8, 2021 21:08

djova force-pushed the djova/mysql-remove-plan-cost branch from abd74ed to 980de7e Compare July 8, 2021 21:08

style

faf33b0

FlorianVeaux approved these changes Jul 9, 2021

View reviewed changes

djova merged commit a3e46fe into master Jul 9, 2021

djova deleted the djova/mysql-remove-plan-cost branch July 9, 2021 14:24

hithwen requested a review from a team July 12, 2021 13:13