Change in 46: `count_all()` expr_fn function now displayed as `count(1)` rather than `count(*)`

[alamb]

This is fallout from #14824

There is a small change in behavior now from DataFusion 45 and DataFusion 46

There were several tests changed (see link below)

In order to preserve behavior from the old version, perhaps we should change count_all() to return count(1) as "count(*)" so it would be consistent with older versions?

Originally posted by @alamb in #14824 (comment)

[alamb]

Since there is a proposed fix in #14824 (comment)

Maybe someone can apply the fix and update the tests ?

[jayzhan211]

I think this is quite different from #14895