On Tue, Jul 1, 2025 at 4:06 PM Nathan Bossart <nathandbossart(at)gmail(dot)com>
wrote:
> rebased
>
> --
> nathan
Everything here makes sense to me, but I do have one question:
In src/bin/pg_upgrade/info.c
@@ -616,11 +630,21 @@ process_rel_infos(DbInfo *dbinfo, PGresult *res, void
*arg)
+ if (inplace)
+ tablespace = psprintf("%s/%s",
+ os_info.running_cluster->pgdata,
+ PQgetvalue(res, relnum, i_spclocation));
+ else
+ tablespace = PQgetvalue(res, relnum, i_spclocation);
I'm sure it's no big deal, but we've already PQgetvalue() fetched that once
for spcloc, and we're going to fetch it again no matter what the value of
inplace is. Is there a reason to not reuse spcloc?