rdblue commented on code in PR #9359:
URL: https://github.com/apache/iceberg/pull/9359#discussion_r1439975347
##########
spark/v3.4/spark/src/main/java/org/apache/iceberg/spark/source/SparkAppenderFactory.java:
##########
@@ -161,7 +162,12 @@ private StructType lazyPosDeleteSparkType() {
}
@Override
- public FileAppender<InternalRow> newAppender(OutputFile file, FileFormat
fileFormat) {
+ public FileAppender<InternalRow> newAppender(OutputFile outputFile,
FileFormat format) {
+ return newAppender(EncryptionUtil.plainAsEncryptedOutput(outputFile),
format);
+ }
+
+ @Override
+ public FileAppender<InternalRow> newAppender(EncryptedOutputFile file,
FileFormat fileFormat) {
Review Comment:
Ah, I was expecting these changes in Spark 3.5. Feel free to move them
there, or we can add them to Spark 3.4 and open follow up PRs.
--
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]
