Hi, Thank you for the patches. They're very important for many extensions.
I've debugged them using simple extensions pg_stat_statements and auto_explain, specifically checking cases involving PlannedStmt and pg_stat_get_activity - , and haven't encountered any issues so far.
However, I have a question: what value should planid have when we execute the standard planner without using a planner_hook? For example, if pg_stat_statementsis disabled, would planid default to zero? If zero is the expected default, should we explicitly write this behavior?
result->planid = 0; -- Best regards, Ilia Evdokimov, Tantor Labs LLC.
