ajantha-bhat commented on issue #10927: URL: https://github.com/apache/iceberg/issues/10927#issuecomment-2285897292
@hantangwangd: Good catch. JSON representation of `schema` already have the `schema-id`. So, I am not sure why we need one more field `schema-id` in the Avro metadata? Also, if we see `partition-spec` it is the JSON representation of the fields of the spec. Not the entire spec. Hence, we need `partition-spec-id`. the `schema-id` field seems irrelevant in the spec. Lets see what others think. -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
