SERVER-82604 recently changed the underlying implementation of StringData to rely on std::string_view, and simply pass through most functionality. However, the string_view::compare API contract is different than StringData, so we would still need to clamp the return values to -1, 0, 1 to provide the same compatibility.

Instead, we would like to audit the codebase for uses of StringData::compare to ensure that callers are not relying on -1/1 values specifically, but instead checking for negative/positive values.