kevinjqliu commented on PR #2538: URL: https://github.com/apache/iceberg-rust/pull/2538#issuecomment-4624251631
> Interestingly the [spec](https://iceberg.apache.org/spec/#appendix-c-json-serialization) makes it seem like this is legitimate both with and without the space? It would appear that if other consumers are failing here, they're not compliant? i feel like thats a formatting issue on the spec and not intention to have both the form with and without the space. That said, the general principle with ser/de for iceberg project is [Postel’s Law](https://lawsofux.com/postels-law/#:~:text=Be%20liberal%20in%20what%20you,conservative%20in%20what%20you%20send.) > Be liberal in what you accept, and conservative in what you send. This is a great place to clarify the spec 😄 thanks for catching it! -- 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]
