scovich commented on PR #10015: URL: https://github.com/apache/arrow-rs/pull/10015#issuecomment-4633590679
> I'm not completely sure what I think of [forcibly annotating all FLBA(16) as UUID]... let me chew on that and report back. But don't block on me I fail to report back by EOD tomorrow. If I'm not mistaken, our decimal code already does some similarly opinionated things with physical types (forcibly converting to a different type than expected, which variant's type canonicalization code actually has to work around), so there would at least be precedent? And injecting the UUID extension type is not worse than today, where we silently treat it as UUID. So it seems like this is a reasonable step forward for now. If we later decide we _really_ want the ability to blow up when somebody forgot to annotate their FLBA(16), we can always do the work to plumb field info through and tighten up the behavior. @alamb - how does that sound to you? -- 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]
