rdblue commented on issue #1812: URL: https://github.com/apache/iceberg/issues/1812#issuecomment-733142448
Those two code sections aren't very related. The write is using an array because the manifests are processed in parallel. That happens to ensure that each manifest has metadata. If the `ManifestFile` that will be written doesn't already have metadata about its contents, then it will be scanned to generate that metadata. On the read side, all of the `ManifestFile` instances in the manifest list are read into memory for later use in scan planning and other operations. I don't see a strong need for those to be in a linked list, but I don't think it would be a problem either. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
