> On Jul 31, 2026, at 19:22, Alberto Piai <[email protected]> wrote: > > Hi, > > On Wed Jul 29, 2026 at 7:07 PM CEST, Sami Imseih wrote: >> Attached is a v12 with these minor fixes. I also improved the commit >> message for 0002. > > A small comment on v12 0001: the patch adds a parameter to > generateClonedIndexStmt() to pass the index name. Is there a reason for > setting the index name in the function, rather than setting it in > DefineIndex() directly on the newly generated statement? >
Good point. I think you are right. We don’t need to touch generateClonedIndexStmt(), we can just set the index name directly in DefineIndex() when needed. PFA v13: addressed Alberto’s comment on 0001; 0002 is the same as v12. Best regards, -- Chao Li (Evan) HighGo Software Co., Ltd. https://www.highgo.com/
v13-0001-tablecmds-preserve-partition-index-names-during-.patch
Description: Binary data
v13-0002-tablecmds-preserve-replica-identity-and-cluster-.patch
Description: Binary data
