wzx140 commented on PR #3523: URL: https://github.com/apache/fluss/pull/3523#issuecomment-4873512583
> @wzx140 Thanks for the pr. IIUC, `paimon.path` is intended to customize the physical location only when the Paimon table is created. Allowing it to be changed after the table has already been created looks unsafe to me, because changing this option does not migrate the existing Paimon metadata/data from the old path to the new one. > > The current check only rejects changing `paimon.path` while `table.datalake.enabled` is true. However, a table can be enabled once, then disabled later, while the Paimon table still exists. In that state, altering `paimon.path` would still be allowed. Could we make `paimon.path` a create-time-only option, or reject changing it once the lake table has ever been created? Thanks for the review! Good catch. I’ll update the implementation accordingly. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
