You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2020/06/29 16:58:03 UTC

[GitHub] [beam] JozoVilcek edited a comment on pull request #12064: [BEAM-10284] Add option to pass configuration into ParquetIO.Sink

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