The following method defined on CanonicalQuery does not take into account whether Query Settings have or haven't been set on a particular query. This could lead to scenarios in which the query hash is wrongfully generated as a SBE one, rather than a classic one, potentially leading to undesired and hard to debug scenarios.

CanonicalQuery::QueryShapeString CanonicalQuery::encodeKey() const {
    return (!_forceClassicEngine && _sbeCompatible) ? canonical_query_encoder::encodeSBE(*this)
                                                    : canonical_query_encoder::encodeClassic(*this);
}