zeroshade commented on PR #34734: URL: https://github.com/apache/arrow/pull/34734#issuecomment-1492555429
I don't think this resolves the issue unfortunately, which is why the existing `GetScalar` method is in the Scalar package. Even if you put the `Scalar` interface into the `arrow` package, if you try to put `GetScalar` as a member of the `Array` objects you'll still run into the issue of trying to actually construct your `Scalar` objects. You'll still run into the cyclical import if you try to call any of the `MakeScalar` methods. So I don't think that this is really worthwhile since it doesn't solve the cyclical import problem. -- 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]
