anoopj commented on code in PR #15049:
URL: https://github.com/apache/iceberg/pull/15049#discussion_r2691284978
##########
api/src/main/java/org/apache/iceberg/FileContent.java:
##########
@@ -18,11 +18,21 @@
*/
package org.apache.iceberg;
-/** Content type stored in a file, one of DATA, POSITION_DELETES, or
EQUALITY_DELETES. */
+/**
+ * Content type stored in a file.
+ *
+ * <p>For V1-V3 tables: DATA, POSITION_DELETES, or EQUALITY_DELETES.
+ *
+ * <p>For V4 tables: DATA, POSITION_DELETES, EQUALITY_DELETES, DATA_MANIFEST,
or DELETE_MANIFEST.
+ */
public enum FileContent {
DATA(0),
POSITION_DELETES(1),
- EQUALITY_DELETES(2);
+ EQUALITY_DELETES(2),
Review Comment:
I had to make this API change so that TrackingInfo can return the
contentType. This is backward compatible since this is a new enum value. If we
don't want to make any API changes, I will need to change
`TrackingInfo.contentType()` to return an `int` instead.
--
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]
