stefan-grafberger commented on code in PR #635: URL: https://github.com/apache/parquet-format/pull/635#discussion_r4458345396
########## LogicalTypes.md: ########## @@ -743,14 +745,15 @@ are: equality comparison and not otherwise interpreted. `checksum` applies to the resolved bytes, except for `ETAG`, which is the -object-store eTag for the whole file referenced by `uri`. +object-store eTag for the whole file referenced by `uri`. If `inline` and `uri` are both +set, `checksum` describes the source at `uri` as it was when `inline` was copied from it. +Like the locator, it is then provenance only. ##### inline The referenced bytes stored inline in the value. If `inline` is set, it supplies the -bytes and any locator fields (`uri`, `offset`, `size`) that are set record where those -bytes came from. A reader may resolve the value from `inline` or from the locator, -whichever suits it. +bytes and any locator fields (`uri`, `offset`, `size`) that are set are provenance +only. Review Comment: It means the locator records where the inline bytes came from, but is not used to resolve the value: when inline is set, a reader returns the inline bytes and does not fetch from uri. An engine or user can still use the locator for other things, e.g. lineage, or to deliberately refresh the inline copy from its source, but that is a separate operation, not reading the FILE value. -- 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]
