Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG]: Incorrect logical type for TimeSpan #533

Open
wldevries opened this issue Jul 19, 2024 · 0 comments
Open

[BUG]: Incorrect logical type for TimeSpan #533

wldevries opened this issue Jul 19, 2024 · 0 comments

Comments

@wldevries
Copy link

Library Version

4.24.0

OS

Windows

OS Architecture

64 bit

How to reproduce?

  1. Create a table with a DataFieldTimeSpan column
  2. Add TimeSpan values larger than 24hours
  3. Parse Parquet file using Pandas

-> ValueError: hour must be in 0..23

I've opened the parquet file using the floor tool and the column has type INT32, converted type TIME_MILIS and logical type TIME (unit: MILLIS, isAdjustedToUTC: True).

A TimeSpan is a period of time and not a time of day like the new TimeOnly type. It seems to me TIME is not the correct logical type for TimeSpan.

Failing test

No response

@wldevries wldevries changed the title [BUG]: Incorrect data type for TimeSpan [BUG]: Incorrect logical type for TimeSpan Jul 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant