razeghi71 commented on issue #8262: URL: https://github.com/apache/arrow-datafusion/issues/8262#issuecomment-1850768997
This problem has been introduced in https://github.com/apache/arrow-datafusion/commit/c9330bc09e583734aed721a208a0684118b170f1 which changed the default empty value for lower and upper bound from `ScalarValue::Null` to `ScalarValue::try_from(field.data_type())`. which is not implemented for map, to fix it we have to either implement it for `Map` or fallback on `Null` value. I have done these two in two draft PRs I have above. In the first one I tried to implement the empty value for map as described in the description of Map. and in the second one I fall back to `Null`. Would be great to get some feedback to see what you folks think? And what is the good approach here? -- 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]
