Add API changes for statistics information in table metadata
API for https://github.com/apache/iceberg/pull/4945 Implementation PR: https://github.com/apache/iceberg/pull/4741
Extracted from https://github.com/apache/iceberg/pull/4741 per https://github.com/apache/iceberg/pull/4741#discussion_r895213091
@rdblue thank you for your review. Updated accordingly.
Currently, based on https://github.com/apache/iceberg/pull/4978
@rdblue updated this to have one stats file for a table.
Per offline ask from @rdblue , I extracted prep change here: https://github.com/apache/iceberg/pull/5129
Updated after merge of https://github.com/apache/iceberg/pull/4945.
@rdblue can you please take a look?
@rdblue thank you for a careful review. Comments applied, the code is much better now. The potentially immature changes to TableMetadata are moved to https://github.com/apache/iceberg/pull/5450
Can you please take another look?
Updated to apply https://github.com/apache/iceberg/pull/5450#discussion_r939727956.
Build failure
> Could not resolve all files for configuration ':classpath'.
> Could not resolve com.palantir.baseline:gradle-baseline-java:4.0.0.
Required by:
project :
> Could not resolve com.palantir.baseline:gradle-baseline-java:4.0.0.
> Could not get resource 'https://plugins.gradle.org/m2/com/palantir/baseline/gradle-baseline-java/4.0.0/gradle-baseline-java-4.0.0.pom'.
> Could not GET 'https://plugins.gradle.org/m2/com/palantir/baseline/gradle-baseline-java/4.0.0/gradle-baseline-java-4.0.0.pom'.
> The server may not support the client's requested TLS protocol versions: (TLSv1.2, TLSv1.3). You may need to configure the client to allow other protocols to be used. See: https://docs.gradle.org/7.4.2/userguide/build_environment.html#gradle_system_properties
> Received fatal alert: handshake_failure
looks unrelated.
Let me push amended commit to restart the build.
Currently, depends on https://github.com/apache/iceberg/pull/5450
Currently, depends on #5450
rebased after that one is merged.
@rdblue please take a look
@findepi, this is looking close. I think we just need to finalize the UpdateStatistics methods. Do you have a PR for the UpdateStatistics implementation? We should be able to work on that in parallel.
I have applied the requested changes. @rdblue please take another look
Thank you for the merge!