fix: datatype_is_logically_equal for dictionaries

[dd-annarose]

Which issue does this PR close?

When checking logical equivalence with Dictionary<_, Utf8> and Utf8View, the response was false which is not what we expect (logical equivalence should be a transitive property).

What changes are included in this PR?

This PR introduces a test and a fix. The test fails without the fix. The fix is simply calling datatype_is_logically_equal again on the v1 and othertype when called with Dictionary<K1, V1> and othertype.

Are these changes tested?

Yes.

Are there any user-facing changes?

No.

[notfilippo]

Makes sense to me! Thanks :)

[gabotechs]

Looks good! thanks @dd-annarose

[dd-annarose]

does anyone know why the [Rust / linux build test (pull_request)](https://github.com/apache/datafusion/actions/runs/21708868623/job/62609567530?pr=20153) check is stuck? it's been like this since the first version of the PR :sad: @Jefffrey maybe?

[dd-annarose]

(trying to retrigger something sorry for the noise everyone)

[Jefffrey]

No idea; maybe we missed our daily prayers to the GitHub Actions gods? Tried rerunning that specific job to no avail 🙁

[Jefffrey]

Seeing if merging up from main might help 🤔

[Jefffrey]

• feat: placeholders in execution plans #20169

Seems to be having same problem; looks like it perhaps is running but we can't get the logs somehow? 🤔

[Jefffrey]

We did recently change the runner to an AWS runner:

• Runs-on for linux-build-lib and linux-test (2X faster CI) #20107

I guess it's related to that if it's only this job being blocked; cc @blaginin

[blaginin]

lets see if it'll get picked up 👀

[blaginin]

seems like an issue on the asf infra side, pinged the team (cc @gmcdonald), let me know if you need to merge this PR urgently

[blaginin]

Hey, I think I now understand the issue. We have a bug where reruns of the action don't work for some reason. The author of this PR hasn't committed to datafusion yet, so the action requires manual approval - and the first run is always cancelled.

I'm trying to figure it out. In the meantime, here's the working CI for this PR: https://github.com/apache/datafusion-sandbox/actions/runs/21755022756

Can confirm the CI is fully green ✅

[gabotechs]

Can confirm the CI is fully green ✅

Nice, is it fine if we merge this now then?

[blaginin]

good by me!

[gabotechs]

In it goes