search

apache / parquet-java

Apache Parquet Java
https://parquet.apache.org/
Apache License 2.0
2.65k stars 1.41k forks source link

MINOR: Support int value for timeOfDay in ProtoTimeConverter #3034

Closed mwong38 closed 1 week ago

mwong38 commented 1 month ago

Rationale for this change

TimeOfDay can sometimes be encoded as INT32 rather than INT64, especially if saved from Kafka Connect.

What changes are included in this PR?

Add support for Int as well as Long.

Are these changes tested?

Yes

Are there any user-facing changes?

No

Fokko commented 1 week ago

Thanks @mwong38 for working on this, and thanks @wgtmac for the review 🙌