rdblue commented on issue #491: Use relative path for manifest_path and file_path URL: https://github.com/apache/incubator-iceberg/pull/491#issuecomment-540173189 I agree with the approach of making relative paths optional. If a path is a full path, then use it. But if it is a relative path, then add some prefix. I'm not sure that we need forward compatibility. We might want to make this an option that people can turn on with a table property. Then it is possible to opt into this behavior and we will maintain forward compatibility by default. We can also include it in a format v2. We are planning to do a format upgrade for row-level deletes, so we _could_ include a format change like this at the same time.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
