nastra commented on code in PR #6274:
URL: https://github.com/apache/iceberg/pull/6274#discussion_r1036815756
##########
core/src/main/java/org/apache/iceberg/MergingSnapshotProducer.java:
##########
@@ -758,20 +758,6 @@ protected Map<String, String> summary() {
return summaryBuilder.build();
}
- /**
- * Apply the update's changes to the base table metadata and return the new
manifest list.
- *
- * @param base the base table metadata to apply changes to
- * @return a manifest list for the new snapshot.
- * @deprecated Will be removed in 1.2.0, use {@link
MergingSnapshotProducer#apply(TableMetadata,
- * Snapshot)}.
- */
- @Deprecated
- @Override
- public List<ManifestFile> apply(TableMetadata base) {
Review Comment:
This is code that changed this method in #4926. We then restored+deprecated
the old API in #6146 for one minor release and are removing it here now.
Let me know if we'd want to keep those methods, then we can just drop the
`Deprecated` annotations from those
--
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]
