Uh oh!
There was an error while loading. Please reload this page.
Core, Orc: Remove deprecated partition stats read functionality - #14998
Merged
pvary merged 1 commit intoMay 22, 2026
Merged
Conversation
gaborkaszabforce-pushed
the
main_remove_deprecated_partition_stat_read
branch
from
January 13, 2026 14:00
0754269 to
b3de566Comparegaborkaszabforce-pushed
the
main_remove_deprecated_partition_stat_read
branch
from
February 2, 2026 13:32
b3de566 to
83d9fe9Comparegaborkaszab
commented
Feb 2, 2026
| testTable.updatePartitionStatistics().setPartitionStatistics(result).commit(); | ||
| assertThat(result.snapshotId()).isEqualTo(currentSnapshot.snapshotId()); | ||
| @Test | ||
| public void testAppendWithBothNullOptionalFields() { |
ContributorAuthor
There was a problem hiding this comment.
These tests are from the removed PartitionStats to keep coverage
gaborkaszabforce-pushed
the
main_remove_deprecated_partition_stat_read
branch
from
February 2, 2026 13:57
83d9fe9 to
3511127CompareThis pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@iceberg.apache.org list. Thank you for your contributions. |
This pull request has been closed due to lack of activity. This is not a judgement on the merit of the PR in any way. It is just a way of keeping the PR queue manageable. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time. |
This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@iceberg.apache.org list. Thank you for your contributions. |
gaborkaszabforce-pushed
the
main_remove_deprecated_partition_stat_read
branch
from
April 14, 2026 11:53
3511127 to
0b9a879CompareThis pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@iceberg.apache.org list. Thank you for your contributions. |
gaborkaszabforce-pushed
the
main_remove_deprecated_partition_stat_read
branch
3 times, most recently
from
May 21, 2026 12:03
55e9a7f to
eda2d88Comparegaborkaszab
commented
May 21, 2026
Uh oh!
There was an error while loading. Please reload this page.
gaborkaszabforce-pushed
the
main_remove_deprecated_partition_stat_read
branch
from
May 21, 2026 14:36
eda2d88 to
bedc8a6Comparepvary
approved these changes
May 22, 2026
Uh oh!
There was an error while loading. Please reload this page.
pvary
commented
May 22, 2026
Contributor
Merged to main. |
dramaticlly added a commit
to dramaticlly/iceberg
that referenced
this pull request
Jun 25, 2026
`build.gradle` line 145 sets `oldVersion = "1.11.0"` — revapi compares the current code against the 1.11.0 released baseline. Section keys in `.palantir/revapi.yml` correspond to that baseline, so new breaks accepted during the 1.12.0 dev cycle belong under `"1.11.0"`, not under a not-yet-released `"1.12.0"` section. Evidence in-file: the `"1.10.0"` section already contains entries with justifications like "Removing deprecated code for 1.11.0" — entries added during the 1.11.0 dev cycle when `oldVersion` was `"1.10.0"`. Same shape applies now. This commit: - Renames the section header `"1.12.0"` -> `"1.11.0"`, merging the new entries from PR apache#16449 into the existing 1.11.0 section that was already on upstream/main (the partition-stats entries from PR apache#14998). - Restores the original justifications on the 17 partition-stats entries that were rewritten by an earlier fixup commit (`fb5ddae9b2 Core: Move partition stats revapi entries to 1.12.0`), back to "Removed deprecated functionality for partition stats". - Drops a duplicate `class org.apache.iceberg.PartitionStats` entry that the earlier fixup commit had introduced. Net effect: the cumulative PR diff against upstream/main is now additive only (new entries appended to the existing 1.11.0 section, no churn on existing entries). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
dramaticlly added a commit
to dramaticlly/iceberg
that referenced
this pull request
Jul 22, 2026
`build.gradle` line 145 sets `oldVersion = "1.11.0"` — revapi compares the current code against the 1.11.0 released baseline. Section keys in `.palantir/revapi.yml` correspond to that baseline, so new breaks accepted during the 1.12.0 dev cycle belong under `"1.11.0"`, not under a not-yet-released `"1.12.0"` section. Evidence in-file: the `"1.10.0"` section already contains entries with justifications like "Removing deprecated code for 1.11.0" — entries added during the 1.11.0 dev cycle when `oldVersion` was `"1.10.0"`. Same shape applies now. This commit: - Renames the section header `"1.12.0"` -> `"1.11.0"`, merging the new entries from PR apache#16449 into the existing 1.11.0 section that was already on upstream/main (the partition-stats entries from PR apache#14998). - Restores the original justifications on the 17 partition-stats entries that were rewritten by an earlier fixup commit (`fb5ddae9b2 Core: Move partition stats revapi entries to 1.12.0`), back to "Removed deprecated functionality for partition stats". - Drops a duplicate `class org.apache.iceberg.PartitionStats` entry that the earlier fixup commit had introduced. Net effect: the cumulative PR diff against upstream/main is now additive only (new entries appended to the existing 1.11.0 section, no churn on existing entries). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
dramaticlly added a commit
to dramaticlly/iceberg
that referenced
this pull request
Jul 23, 2026
`build.gradle` line 145 sets `oldVersion = "1.11.0"` — revapi compares the current code against the 1.11.0 released baseline. Section keys in `.palantir/revapi.yml` correspond to that baseline, so new breaks accepted during the 1.12.0 dev cycle belong under `"1.11.0"`, not under a not-yet-released `"1.12.0"` section. Evidence in-file: the `"1.10.0"` section already contains entries with justifications like "Removing deprecated code for 1.11.0" — entries added during the 1.11.0 dev cycle when `oldVersion` was `"1.10.0"`. Same shape applies now. This commit: - Renames the section header `"1.12.0"` -> `"1.11.0"`, merging the new entries from PR apache#16449 into the existing 1.11.0 section that was already on upstream/main (the partition-stats entries from PR apache#14998). - Restores the original justifications on the 17 partition-stats entries that were rewritten by an earlier fixup commit (`fb5ddae9b2 Core: Move partition stats revapi entries to 1.12.0`), back to "Removed deprecated functionality for partition stats". - Drops a duplicate `class org.apache.iceberg.PartitionStats` entry that the earlier fixup commit had introduced. Net effect: the cumulative PR diff against upstream/main is now additive only (new entries appended to the existing 1.11.0 section, no churn on existing entries). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
In 1.12.0 we can remove the deprecated read functionality for partition stats