JozoVilcek edited a comment on pull request #12064: URL: https://github.com/apache/beam/pull/12064#issuecomment-649031901
Hi @iemejia , yes, the PR is to control paqruet settings you mentioned. I was considering a Map based settings but since parquet writer is hadoop based anyway now, I thought there is not reason why not to expose full config. I do not see any reason why not to make it Map if this is something you prefer. Now I think it is probably a better choice ---------------------------------------------------------------- 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]
