Elastic.Clients.Elasticsearch 8.9.2 QueryBuilder must子句查询返回全部数据问题

问题根源 #

在Elastic.Clients.Elasticsearch 8.x版本中，BoolQuery的Must属性要求的类型是ICollection<QueryContainer>，而非你代码中使用的ICollection<Query>。你直接将List<MatchQuery>通过as强制转换为ICollection<Query>赋值给Must，会导致Elasticsearch无法正确识别查询条件——最终执行的是一个无约束的布尔查询，这种查询默认会匹配所有文档，所以你得到了全量返回结果。

另外，当firstName和lastName都为空时，must列表是空的，此时布尔查询同样会返回所有文档，这也是一个需要规避的潜在问题。

修正后的代码 #

public class QueryBuilder
{
    public Query BuildQuery(string firstName, string lastName)
    {
        var must = new List<QueryContainer>();

        if (!string.IsNullOrWhiteSpace(firstName))
        {
            must.Add(new MatchQuery("firstName") { Query = firstName });
        }

        if (!string.IsNullOrWhiteSpace(lastName))
        {
            must.Add(new MatchQuery("lastName") { Query = lastName });
        }

        // 无有效条件时返回空结果查询，避免全量返回
        if (!must.Any())
        {
            return new MatchNoneQuery();
        }

        return new BoolQuery
        {
            Must = must
        };
    }
}

关键修正点 #

1. 类型匹配：将List<MatchQuery>替换为List<QueryContainer>，QueryContainer是Elastic.Clients.Elasticsearch中专门用于封装各类查询的容器类，所有具体查询类型（如MatchQuery）都能隐式转换为QueryContainer，满足BoolQuery.Must的类型要求。
2. 空条件处理：新增判断逻辑，当没有任何有效查询条件时返回MatchNoneQuery，确保不会意外返回所有文档。
3. 移除不安全转换：直接将must列表赋值给BoolQuery.Must，避免类型转换导致的解析错误。

内容的提问来源于stack exchange，提问作者SimonD