sql: udf logic tests not to skip dropping databases

[chengxiong-ruan]

Now that we support drop functions in legacy schema changer
and declarative schema changer falls back to legacy when seeing
any function descriptor, we can move udf logic tests back
to normal configs.

Release note: None.
Release justification: test only change.

[cockroach-teamcity]

This change is 

[ajwerner]

🎉

Reviewed 9 of 24 files at r1, all commit messages.
Reviewable status: complete! 1 of 0 LGTMs obtained (waiting on @ajwerner)

[chengxiong-ruan]

pkg/sql/opt/exec/execbuilder/tests/fakedist/BUILD.bazel line 12 at r1 (raw file):

        "//pkg/sql/opt/exec/execbuilder:testdata",  # keep
    ],
    shard_count = 2,

dev generate generated this change. My guess is that we were at the edge of some threshold on the number of tests.

[ajwerner]

The shards are min(num_tests_in_config, 16)

[chengxiong-ruan]

The shards are min(num_tests_in_config, 16)

heh. I made a mistake, I should have just used local config for executor tests instead of removing the config from the test file which trigger all these changes :(

[chengxiong-ruan]

udf expressions are built into new RoutineExprs. For now we need to skip DistSQL for any RoutineExpr before we plumb a function resolver into DistSQL or maybe this todo can be done.

[chengxiong-ruan]

TFTR!
bors r+

[craig]

Build failed (retrying...):

• Bazel Essential CI (Cockroach)

[craig]

Build succeeded:

• Bazel Essential CI (Cockroach)