CTTY commented on issue #2036: URL: https://github.com/apache/iceberg-rust/issues/2036#issuecomment-3775312702
An enum seems like a good idea. Listing out some potential alternatives from the top of my mind: - Add encryption-aware `FileWrite` and `FileRead` implementations - Push encryption logic fully to the upcoming `Storage` trait this design decision also depends on how we plan to create `Input`/`OutputFile`. Do you think we will have an `EncryptionManager` in rust and where does that live? -- 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]
