Hello!
> In your patch, only the existence check is moved to basic_archive_configured().
> Would it also make sense to move the filename length check there? If so,
> we could potentially remove the check_archive_directory GUC check hook entirely.
Indeed.
I'm not sure, do I need to keep the check_archive_directory declaration for backport branches? I will split the patch
intotwo parts.
regards, Sergei