Remove repeated setTag

[sohpeach]

This is already set in getWithoutFallback

[ask-bonk]

The PR's claim is confirmed. At line 290, getWithoutFallback already sets readSpan.setTag("cache_hit"_kjc, result != kj::none) — which evaluates to false when there's no cache entry. The removed line at 553 (previously line 553 in the old code) was redundantly setting readSpan.setTag("cache_hit"_kjc, false) right after getWithoutFallback already returned kj::none (the else branch of KJ_IF_SOME).

This is a correct, trivial cleanup with no behavioral change.

LGTM

github run