pitrou commented on PR #36846: URL: https://github.com/apache/arrow/pull/36846#issuecomment-1680794852
> I'm not interested in these special cases at least, it is just in here because the original author did this, but I'm also fine with only including the obvious and getting rid of all the flags. Well, we shouldn't get rid of existing functionality. So any cast that is already supported should keep being supported (unless it's a glaring mistake of course!). As for getting rid of all the flags, I don't know if the options need to be as fine-grained as you made them, I thought you had a use for that :-) That said, it's nice to make things configurable, and it seems you did most of the work anyway? @westonpace What do you think? -- 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]
