danielcweeks commented on PR #11067: URL: https://github.com/apache/iceberg/pull/11067#issuecomment-2414890266
@ajantha-bhat I feel like this has bounced back and forth on the levels, but we should be focused on getting the structure correct first and then addressing the ToC. This current version has `Specification` at the same level as its children, which is not correct. L2/H2 should be: ``` ## Format Versioning ## Goals ## Overview ## Specification ## Appendix A: Format-specific Requirements ## Appendix B: 32-bit Hash Requirements ## Appendix C: JSON serialization ## Appendix D: Single-value serialization ## Appendix E: Format version changes ## Appendix F: Implementation Notes ``` Everything else should be nested below. -- 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
