[GitHub] [parquet-mr] wgtmac commented on pull request #1142: PARQUET-1647: Add logical type FLOAT16

["wgtmac (via GitHub)" <gi...@apache.org>]

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-unsubscribe@parquet.apache.org

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