milleruntime commented on issue #2001: URL: https://github.com/apache/accumulo/issues/2001#issuecomment-832180952
I was able to discuss with @ctubbsii a design alternative to the minimal approach I was taking. The idea we came up with was to add a factory layer (something like CryptoServiceFactory) to handle the table -> CryptoService association. The file utilities that handle reading and writing to RFiles will still be ignorant of their table but just know whether a file is encrypted or not. Either way I think we will have to rename the properties to use the `general` prefix. -- 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. For queries about this service, please contact Infrastructure at: [email protected]
