[GitHub] [arrow] ggershinsky commented on pull request #9631: ARROW-11644: [Python][Parquet] Low-level Parquet decryption in Python

[GitBox <gi...@apache.org>]

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