JingsongLi commented on PR #9710: URL: https://github.com/apache/paimon/pull/9710#issuecomment-5611751085
Could we pass the concrete target partition path instead of using `path: null` as a reset operation? The writer already knows its output directory. The REST server could derive and normalize the default partition path from the table metadata and partition spec, then remove the stored path override when the requested path equals that default. Whether the server stores this as NULL should be an implementation detail. The API semantics would be: - Omit `path`: keep the current location. - Provide `path`: specify the desired location. For this PR, we could limit changes to existing locations to switching back to the default directory. Such a change should still require replacement statistics for the same partition and apply both atomically; an ordinary `createPartitions(ignoreIfExists=true)` should not silently relocate an existing partition. This would also require recognizing the default path before the custom-location overlap/conflict checks. Passing a non-null path alone does not provide compatibility with older servers, so the updated server contract would still need to be explicit. -- 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]
