wgtmac commented on PR #1142:
URL: https://github.com/apache/parquet-mr/pull/1142#issuecomment-1726797426

   > CI failures are likely due to the fact that the addition of the logical 
type to parquet-format is unmerged, so the specific [PR 
branch](https://github.com/apache/parquet-format/pull/184) needs to be manually 
installed for the build to pass. I'm not sure if there's a good solution yet, 
as this implementation needs to be present for said parquet-format PR to be 
voted on and merged.
   
   I agree that this is not elegant. For now, we can only review the 
functionality and completeness of the current implementation. Once the format 
change is submitted (and released), we need to take another pass on this PR.


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

To unsubscribe, e-mail: dev-unsubscr...@parquet.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to