rdblue commented on code in PR #8252:
URL: https://github.com/apache/iceberg/pull/8252#discussion_r1506325920
##########
api/src/main/java/org/apache/iceberg/io/PositionOutputStream.java:
##########
@@ -29,4 +29,15 @@ public abstract class PositionOutputStream extends
OutputStream {
* @throws IOException If the underlying stream throws IOException
*/
public abstract long getPos() throws IOException;
+
+ /**
+ * Return the stored length of the output object. Can differ from the
current position for
+ * encrypting streams, and other non-length-preserving streams.
+ *
+ * @return output storage object length in bytes
Review Comment:
I think this should be the current stored length in bytes, since it will be
called during a write.
--
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: issues-unsubscr...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
