Hi

Unfortunately patch does not apply due recent commits. Any chance this can be 
fixed (and even committed in pg12)?

>>  And few questions:
>>  - reindexdb has concurrently flag logic even in reindex_system_catalogs, 
>> but "reindex concurrently" can not reindex system catalog. Is this expected?
>
> If support is ever added, then reindexdb supports it automatically. It
> seems simpler to not have to repeat the same checks in two places.

ok, reasonable for me

>>  - psql/tab-complete.c vs old releases? Seems we need suggest CONCURRENTLY 
>> keyword only for releases with concurrently support.
>
> It seems we don't do version checks for tab completion of keywords.

Hmm, yes, i found only few checks for "create trigger" syntax about "EXECUTE 
FUNCTION"/"EXECUTE PROCEDURE" difference in 11

regards, Sergei


Reply via email to