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

Posted to dev@parquet.apache.org by GitBox <gi...@apache.org> on 2021/04/20 12:43:27 UTC

[GitHub] [parquet-mr] Miksu82 commented on pull request #895: Parquet-2030: Expose page size row check configurations to ParquetWri…

Miksu82 commented on pull request #895:
URL: https://github.com/apache/parquet-mr/pull/895#issuecomment-823242208


   > We do not need to validate the whole feature of the page size check only the fact that the properties have passed properly
   
   Yeah you are right, that should be in `InternalParquetRecordWriter` unit tests. I'll try to simplify the tests I've added so that they only test that the property has been set.


-- 
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