bryanck commented on PR #10173: URL: https://github.com/apache/iceberg/pull/10173#issuecomment-2233789778
@Dawnpool I'd prefer to keep Hadoop/HDFS specific code out of the sink, beyond loading generic properties as we're doing now. IIRC you can set the user principal via an environment variable. I have [this branch](https://github.com/bryanck/iceberg/tree/kc-runtime) I'm working on to add in the runtime distribution if you want to try that to test. You might want to investigate how the Flink sink handles this without special properties. If it turns out we do need it, I feel like it might belong in HadoopFileIO, rather than here. -- 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
