search

datafusion-contrib / datafusion-functions-json

JSON / JSONB support for DataFusion (unofficial)
https://crates.io/crates/datafusion-functions-json
Apache License 2.0
13 stars 5 forks source link

invoke: allow `LargeUtf8` #19

Closed davidhewitt closed 1 week ago

davidhewitt commented 1 week ago

tests seem to cover this case, but it still seems prudent to add to the match for completeness?

davidhewitt commented 1 week ago

It seems to me that existing tests do create LargeUTF8 JSON data, but maybe if invoking the function as a unit test?

davidhewitt commented 1 week ago

Ok great, I worked out how to add a test, which did catch one case. Will proceed to merge now.