> On Wed, Mar 19, 2025 at 11:38 AM Sami Imseih <samims...@gmail.com> wrote: > > ... as I made the hook signature match that of > > ParseExplainOptionList, so both pstate and the options list > > are now available to the hook. > > This version LGTM, except it's not pgindent-clean.
ugh, sorry about that. ran it for both modified files and it found one indentation correction from v2. > index 1d4be3c18ac..8fe1ca5c73e 100644 38c38 < + (*explain_validate_options_hook) (es, options, pstate); --- > + (*explain_validate_options_hook)(es, options, pstate); 59c59 < 2.47.1 v3 attached. -- Sami Imseih Amazon Web Services (AWS)
v3-0001-Add-new-hooks-for-performing-additional-EXPLAIN-o.patch
Description: Binary data
