Error NamedStructField should be rewritten in OperatorToFunction with subquery if query is wrapped in view

[ahirner]

Describe the bug

When selecting from a view that selects fields from structs, this error is thrown:

Internal error: NamedStructField should be rewritten in OperatorToFunction.

To Reproduce

In datafusion 38.0.0 cli:

> create or replace view v as select struct(1, 'b')['c1'];
0 row(s) fetched.
Elapsed 0.001 seconds.

> select * from v;
Internal error: NamedStructField should be rewritten in OperatorToFunction.
This was likely caused by a bug in DataFusion's code and we would welcome that you file an bug report in our issue tracker

Expected behavior

+--------------------------------+
| struct(Int64(1),Utf8("b"))[c1] |
+--------------------------------+
| b                              |
+--------------------------------+
1 row(s) fetched.

Additional context

It errors the same when a subquery is viewed: create view v as select (select struct(1, 'b')['c1']);.
This error appeared once as a regression in 37.0: #10029

[jonahgao]

It appears to be successful on the current main branch.

DataFusion CLI v38.0.0
> create or replace view v as select struct(1, 'b')['c1'];
0 row(s) fetched.
Elapsed 0.005 seconds.

> select * from v;
+--------------------------------+
| struct(Int64(1),Utf8("b"))[c1] |
+--------------------------------+
| b                              |
+--------------------------------+
1 row(s) fetched.
Elapsed 0.006 seconds.

[sergiimk]

Also hit a similar error in a different scenario and can confirm that master version fixed it for me.

[ahirner]

Can confirm that it's gone for our more complicated use case in 8fcb3e4b, thus also probably in 0.39.
Was or should it be covered in a test? Otherwise feel free to close.