On Tue, 1 Nov 2022 at 23:56, Simon Riggs <simon.ri...@enterprisedb.com> wrote:
> > I haven't checked the rest of the patch, but +1 for allowing VACUUM FULL > > within a user txn. > > My intention was to prevent that. I am certainly quite uneasy about > changing anything related to CLUSTER/VF, since they are old, complex > and bug prone. > > So for now, I will block VF, as was my original intent. > > I will be guided by what others think... so you may yet get your wish. > > > > Maybe the error message needs to be qualified "...when multiple > > relations are specified". > > > > ERROR: VACUUM cannot run inside a transaction block > > Hmm, that is standard wording based on the statement type, but I can > set a CONTEXT message also. Will update accordingly. > > Thanks for your input. New version attached, as described. Other review comments and alternate opinions welcome. -- Simon Riggs http://www.EnterpriseDB.com/
single_table_vacuum.v2.patch
Description: Binary data
