When we try to copy slots and the wal_level on the new server is minimal, we error out, so shouldn't we do the same here and error_out if track_commit_timestamp is not enabled and we have some valid commit_ts data to copy?
+1 Sounds reasonable to me. It's better to give an explicit error; otherwise, we would remove data that the user clearly wants to migrate to the new cluster. And deleting data just because the user forgot to specify one parameter in a new cluster looks like a bad joke to me.