rdblue commented on pull request #2096:
URL: https://github.com/apache/iceberg/pull/2096#issuecomment-772102052


   > Do you have any suggestion on if we want to add schema id to the history 
table?
   
   I don't think that we want to add it to history. The snapshot log should 
refer to snapshots by ID and we can get the schema easily enough from there. 
For the actual metadata table, we don't want to expose schema ID because that's 
something that users should ideally never need to know much about. We've 
avoided adding IDs in other similar situations. For example, you can rewrite 
manifests for a specific partition spec, by ID. But we didn't include that in 
the stored procedure because we haven't exposed the partition spec ID anywhere 
to SQL users.


----------------------------------------------------------------
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]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to