<div dir="ltr"><div>I beg a pardon if that all is trivial to you, I get to this part of postgres for the first time.</div><div><br></div><div>The error around index creation matches the release note changes to index creation.</div><div>But the source for this error that we see is actually an expected fail:</div><div><br></div><div>regress/sql/repack.sql:</div><div>-- This will fail. Silence the message as it's different across PG versions.</div><div>SET client_min_messages = fatal;</div><div>CREATE UNIQUE INDEX CONCURRENTLY idx_badindex_n ON tbl_badindex (n);</div><div>SET client_min_messages = warning;</div><div><br></div><div>When diving into the diff of result vs expected output I found some issues on the path used for the tablespace.</div><div>I needed to add this in my repro iteration</div><div> $ rm -rf /tmp/pg-repack-tablespace/*</div><div><br></div><div>The diff then comes down to:</div><div>- whitespace</div><div>- the explicit listing of TABLESPACE pg_default in some statements</div><div><br></div><div>The output of those two calls has changed:</div><div>  repack.repack_indexdef(indexrelid, 'testts1'::regclass, NULL, true)</div><div>If there is a value instead of NULL the output did not change.</div><div><br></div><div>This argument is defined as</div><div>* @param»··tablespace»·Namespace for the index. If NULL keep the original.</div><div><br></div><div>In the cases with a tablespace already defined as "foo" the expected output already lists "TABLESPACE foo" that is why it is not changing.</div><div>The cases with NULL had nothing about it before but now get "TABLESPACE pg_default" which exactly matches the defintion of the argument.</div><div><br></div><div>Upstream has several changes like [1] to add more accepted output versions.</div><div><br></div><div>I think that it might be ok to integrate a new pg-repack-1.3.4/regress/expected/tablespace_2.out with the new content.</div><div>I'm gonna suggest upstream at pg_repack as well, but I'd be happy to hear your expertise on it as well.</div><div><br></div><div>[1]: <a href="https://github.com/reorg/pg_repack/commit/4002b4e18f030c8e265707e0174b184d325eeb1b">https://github.com/reorg/pg_repack/commit/4002b4e18f030c8e265707e0174b184d325eeb1b</a></div><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><span style="color:rgb(136,136,136);font-size:12.8px">Christian Ehrhardt</span><div style="color:rgb(136,136,136);font-size:12.8px">Software Engineer, Ubuntu Server</div><div style="color:rgb(136,136,136);font-size:12.8px">Canonical Ltd</div></div></div></div></div>
</div>