rdblue commented on code in PR #6762:
URL: https://github.com/apache/iceberg/pull/6762#discussion_r1340726876
##########
api/src/main/java/org/apache/iceberg/encryption/EncryptedOutputFile.java:
##########
@@ -37,4 +37,9 @@ public interface EncryptedOutputFile {
* #encryptingOutputFile()}.
*/
EncryptionKeyMetadata keyMetadata();
+
+ /** Underlying output file for native encryption. */
+ default OutputFile rawOutputFile() {
+ return null;
Review Comment:
Should this throw `UnsupportedOperationException`? That seems like a good
idea to me so that we don't get NPE when trying to use this with Parquet.
--
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]
