Tom Lane wrote: > Bruce Momjian <br...@momjian.us> writes: > > Looking at the pg_upgrade code some more, I found that it was not > > removing the PG_VERSION file when deleting <= 8.4 tablespace files. > > This might confuse administrators so the attached patch adds the removal > > of PG_VERSION. I would like to apply this to master and 9.0.X. > > ... why is that a good idea?
The script already deletes the 8.4 database directories, but leaves PG_VERSION behind. Why keep it when all the 8.4 data is gone? The script also dates PGDATA for 8.4, so there is nothing left pointing to that directory. Here are the delete script file contents after the patch: #!/bin/sh rm -rf /u/pgsql.old/data rm -f /rtmp/pgsql/PG_VERSION rm -rf /rtmp/pgsql/1 rm -rf /rtmp/pgsql/11564 rm -rf /rtmp/pgsql/16384 rm -rf /rtmp/pgsql/16385 rm -rf /rtmp/pgsql/27628 -- Bruce Momjian <br...@momjian.us> http://momjian.us EnterpriseDB http://enterprisedb.com + It's impossible for everything to be true. + -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers