mihaibudiu commented on PR #4026: URL: https://github.com/apache/calcite/pull/4026#issuecomment-2451070225
The "right" way to do this is to use the type information, passing it from the higher levels of the JSON document which know it. The type information should be passed down from toRex() through sargFromJson. Today it's not, and these methods are public, so probably a new version of these methods (sargFromJson, rangeSetFromJson, rangeFromJson, rangeEndpointFromJson) should be created which takes a type as an extra argument. -- 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]
