singhpk234 commented on code in PR #13936:
URL: https://github.com/apache/iceberg/pull/13936#discussion_r2310853943


##########
format/spec.md:
##########
@@ -1861,6 +1861,16 @@ Java writes `-1` for "no current snapshot" with V1 and 
V2 tables and considers t
 
 Some implementations require that GZIP compressed files have the suffix 
`.gz.metadata.json` to be read correctly. The Java reference implementation can 
additionally read GZIP compressed files with the suffix `metadata.json.gz`.  
 
+### Schema evolution and writing with old schemas
+
+Writers must write out all fields with the types specified from the table 
schema. Writers should use the latest schema for writing. Not writing out all 
columns or not using the latest schema can change the semantics data written. 
The following are possible inconsistencies that can be introduced:

Review Comment:
   [minor] 
   > all fields with the types specified from the table schema
   
   is it worth explicitly clarifying that this can be any schema as saying 
table schema might mean latest schema, from my understanding the intention is 
MUST is only for writing all the columns, using latest schema is good to have. 



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

Reply via email to