Commit 1bec9cf
authored
fix: incorrect descriptions of snapshot operations (#2513)
…unctions.
<!--
Thanks for opening a pull request!
-->
<!-- In the case this PR will resolve an issue, please replace
${GITHUB_ISSUE_ID} below with the actual Github issue id. -->
<!-- Closes #${GITHUB_ISSUE_ID} -->
# Rationale for this change
Currently, the documentation of these write functions contains some
inaccuracies regarding snapshot operations. These update methods do not
actually produce `replace` snapshots, but rather `overwrite` snapshots,
because the rewrite here modifies the data (dropping the records that
match the overwrite filter).
A `replace` snapshot is only generated when the table data itself is not
changed, such as during data file rewrites or manifest rewrites.
https://iceberg.apache.org/spec/#snapshots
<img width="1536" height="418" alt="image"
src="https://github.com/user-attachments/assets/a96b0e1c-86af-45da-98e5-58753e7f7482"
/>
## Are these changes tested?
## Are there any user-facing changes?
<!-- In the case of user-facing changes, please add the changelog label.
-->1 parent cff0c64 commit 1bec9cf
1 file changed
+3
-3
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
569 | 569 | | |
570 | 570 | | |
571 | 571 | | |
572 | | - | |
| 572 | + | |
573 | 573 | | |
574 | 574 | | |
575 | 575 | | |
| |||
629 | 629 | | |
630 | 630 | | |
631 | 631 | | |
632 | | - | |
| 632 | + | |
633 | 633 | | |
634 | 634 | | |
635 | 635 | | |
| |||
1396 | 1396 | | |
1397 | 1397 | | |
1398 | 1398 | | |
1399 | | - | |
| 1399 | + | |
1400 | 1400 | | |
1401 | 1401 | | |
1402 | 1402 | | |
| |||
0 commit comments