fix returning of values vs array

[adriangb]

No description provided.

[adriangb]

From @samuelcolvin:

we're using dict.values() in datafusion-functions-json then just returning that array, I'm a bit concerned that the arrays return are going to be the wrong length (and intend generally wrong), e.g. that they're using the just the unique values, not the full array

I think this should fix it. I wasn't able to get the existing tests to fail, I thought I would be able to by adding a repeated key but couldn't, but this new test does fail on the target branch:

UDF returned a different number of rows than expected. Expected: 7, Got: 4

[samuelcolvin]

A lot of the diff doesn't seem related to this change?

[adriangb]

A lot of the diff doesn't seem related to this change?
Yeah not sure why cargo fmt or clippy did that, but it does seem correct so maybe lets just merge it...

[adriangb]

@samuelcolvin should we merge this?

[samuelcolvin]

thanks