expr.slt test failing in main

[Omega359]

Describe the bug

Running "expr.slt"
External error: query failed: DataFusion error: Schema error: No field named t2."struct(t1.time,t1.load1,t1.load2,t1.host)". Valid fields are t2."named_struct(Utf8(""c0""),t1.time,Utf8(""c1""),t1.load1,Utf8(""c2""),t1.load2,Utf8(""c3""),t1.host)".
[SQL] select t2."struct(t1.time,t1.load1,t1.load2,t1.host)"['c3'] as host, sum(coalesce(t2."struct(t1.time,t1.load1,t1.load2,t1.host)")['c1']) from (select struct(time,load1,load2,host) from t1) t2 where t2."struct(t1.time,t1.load1,t1.load2,t1.host)"['c3'] IS NOT NULL group by t2."struct(t1.time,t1.load1,t1.load2,t1.host)"['c3'] order by host;
at test_files/expr.slt:2290

Seems to be related possibly to an interaction between #9743 and #9871. Seen in CI run on #9878 and locally.

To Reproduce

cargo test --test sqllogictests -- expr

Expected behavior

test passes

Additional context

No response

[jayzhan211]

Why is the error not failed in CI previously 😕 It seems like a non-trivial fix, should we revert the commit in this case?

[alamb]

I agree with @Omega359 that this looks like a logical conflict with #9743

Specifically #9743 causes the column names to be different.

[alamb]

Specifically since #9743 changed the name of the columns, this test would have had to use the new names.

[alamb]

Thank you very much @Omega359 and @jayzhan211 -- sorry for the disruption