rdblue opened a new pull request, #17022: URL: https://github.com/apache/iceberg/pull/17022
Metrics modes are configured in table properties based on column names, but content stats needs to look up `MetricsMode` based on column ID. This adds support to look up metrics mode by ID, `columnMode(int)`, and adds an accessor for the IDs for which there is a metrics mode. This is needed to build a table's current content stats struct for writing. This also cleans up a couple of issues with `MetricsConfig`: * Removes the deprecated `forPositionDelete(Table)` instead of modifying its behavior. Also removes uses of this deprecated method, using `forPositionDelete()` instead. * Updates the deprecation of `fromProperties` to 1.13.0 instead of 2.0.0 because this is in core. Also removes uses of this deprecated method, which should be done when it is deprecated. -- 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]
