AlenkaF commented on PR #41795:
URL: https://github.com/apache/arrow/pull/41795#issuecomment-2147584517
> I know of at least one previous discussion regarding the term "logical
type". This PR may not be the place to do it, but readers that are familiar
with existing arrow implementations may be confused because `Schema.fbs`, while
authoritative, parameterizes types differently than most implementations (e.g.,
pretty much every implementation has an `pyarrow.int32()` not
`pyarrow.int(32)`). I think this only affects `Int`, `Date`, `Time`, and
`FloatingPoint`, and ("types" that are parameterized differently in pyarrow vs.
Schema.fbs.
>
> (I am not sure the best way to incorporate that here or if it's a battle
for another day 🙂 )
Ha. Yes, this is true. Maybe if the title in the table is changed from
"Additional info stored" to "Parameters" it is clearer and then, I guess, it is
up to the implementation to decide if the parameter defines a separate type
(`int32`) or is actually a parameter of the type (`int(bit_width=32,
is_signed=False)`).
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
