ggershinsky commented on pull request #9631:
URL: https://github.com/apache/arrow/pull/9631#issuecomment-831307633


   > I'll try to review soon. I think the rationale presented by the end-user 
makes sense, @ggershinsky are you still very concerned with this?
   
   Well, a couple of points:
   -the comment above is not very accurate: high-level API doesn't perform KMS 
functions; instead, it provides a plug-in interface for any KMS. This end-user 
might be not familiar enough with the high-level API; there is a good chance it 
will fit their needs. In an unlikely event it doesn't, it would be good to get 
a technical feedback that refers to gaps in this API.
   -my position hasn't changed - I'm ok with the read/decryption part of the 
low-level API, as a temp measure before the Python version of the high-level 
API is ready. I am concerned with exposing the write/encryption part of the 
low-level API, for the reasons I've mentioned in the ticket. 


-- 
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:
us...@infra.apache.org


Reply via email to