dannycjones commented on issue #2184:
URL: https://github.com/apache/iceberg-rust/issues/2184#issuecomment-4830644346

   There was some discussion on the Iceberg Slack dev channel 
(https://apache-iceberg.slack.com/archives/C03LG1D563F/p1776937382362429), but 
I'll summarize it here.
   
   The consensus is that this is just a field set for documentation purposes 
and must not be relied on by engines. It was also broadly agreed we should not 
add this field to the footer for compatibility reasons for some specific 
engines. Ultimately, this change would need to be added to every Iceberg 
library and effectively becomes a spec requirement.
   
   This issue does make me wonder if its worth exploring the ability to add 
arbitrary footer metadata to data files, although I expect that's not possible 
given we would support different formats, some of which may not have the 
concept of footer key,value properties. I see that iceberg-java doesn't support 
this today.


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

Reply via email to