Add support for Explain Analyze for Postgres v13#1230
Conversation
mrigger
left a comment
There was a problem hiding this comment.
I think the code is currently used only by the Query Plan Guidance (QPG) component. If we execute ANALYZE for it, performance would drop. I guess we could have separate public methods, one for QPG and one that we use in the provider class.
Sorry, something went wrong.
I see, looking over the |
Sorry, something went wrong.
|
We could add an enum (see ). In this enum, we can specify a newly-created method in the explain generator that can generate the full range ofEXPLAIN, while we can keep using the restricted method for QPG.
|
Sorry, something went wrong.
|
@yaojiejia, I see you have been working on multiple other PRs. Do you think it would be possible to focus on the open ones, so we can merge those first? |
Sorry, something went wrong.
Of course, will prioritize these for now, thanks |
Sorry, something went wrong.
Just included a method for EXPLAIN. Lmk if this is something we are looking for |
Sorry, something went wrong.
Closes 12. EXPLAIN ANALYSE from #912