flyrain commented on PR #12017: URL: https://github.com/apache/iceberg/pull/12017#issuecomment-3340835903
@tomtongue , thanks for replying. The plan sounds good to me. We will need to deprecate `write.metadata.path` before removing it, as it was out already. > I think if users don't specify the location property in a view creation, the view metadata should be located in iceberg-table-path/metadata I don't quite understand this, but `Location` is a `required` field in the view spec, either catalog or engine have to pick one location when it commits the view metadata. One option catalogs usually go with is provide a default location by concatenating namespace location with view name. In short, the view location shouldn't be empty in any case. > users specif the parameter, the view metadata should be located in user-specified-location/ That's a behavior change, as the current behavior is putting metadata under `user-specified-location/metadata`. I'm not sure if we could change this behavior. I'd prefer to keep it as is. Is there a big concern of having `metadata` in the path? -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
