ookumuso commented on PR #11112: URL: https://github.com/apache/iceberg/pull/11112#issuecomment-2418059649
> @ookumuso a couple small remaining comments: > > 1. It looks like we're only using 20 bits, not 24 like in the description > > 2. The aws.md docs don't reflect the updated pathing structure @danielcweeks thanks: 1. Intentionally dropped the bits as it should more than enough, updated the PR description and aws.doc as well 2. Updated the aws.md doc with the new path structure -- 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