batcheval: move remote sst handling to top

[dt]

We don't want to do any other processing if it is a remote sst.

Release note: none.
Epic: none.

[blathers-crl]

It looks like your PR touches production code but doesn't add or edit any test code. Did you consider adding tests to your PR?

_{🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.}

[cockroach-teamcity]

This change is 

[stevendanna]

Works for me. Looks like the main thing this is skipping at the moment is the capacity check which we've seen in profiles account for non-trivial time.

[dt]

main thing this is skipping at the moment

To be pedantic we're skipping a lot in the remote case, like collision checks, but we were already skipping that before this move. This is move it just moving it the rest of the way, to the very top -- swapping places with the capacity check, which doesn't make sense for remote ssts that don't use capacity.

[dt]

TFTRs!

bors r+

[craig]

Build succeeded:

• Bazel Essential CI (Cockroach)