gaborkaszab opened a new pull request, #15057: URL: https://github.com/apache/iceberg/pull/15057
#15045 specifies that query parameters that has an effect on what to be returned in the metadata result, such as `snapshots` should be incorporated into ETag calculation. In practice, loadTable with `snapshots=ALL` shouldn't return the same ETag as loadTable with `snapshots=REFS`. This PR changes the ETag calculation in the reference IRC. Areas to note: 1) createTable, registerTable doesn't send query params and return table metadata as if `snapshots=ALL` was given. It's essential to make sure these give the same ETag as loadTable with `snapshots=ALL`. 2) In general, the order of the params shouldn't matter when calculating ETags. `p1=v1,p2=v2` should give the same ETag as `p2=v2,p1=v1` if the state of the table is the same. -- 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]
