percolate query sorted by "_doc" is incorrectly matching date range query

[davidbkemp]

Elasticsearch version 7.5.1

Plugins installed: []

JVM version 1.8.0_232

OS version Darwin C02XF3HBJHD2 18.7.0 Darwin Kernel Version 18.7.0: Sun Dec 1 18:59:03 PST 2019; root:xnu-4903.278.19~1/RELEASE_X86_64 x86_64

Sorting a percolate search by "_doc" results in range queries being incorrectly matched.

Note: this could be related to #52617

Steps to reproduce:

1. Create an index

PUT /myindex
{
  "mappings": {
    "properties": {
      "d": {
        "type": "date"
      },
      "q": {
        "type": "percolator"
      }
    }
  }
}

2. Index a couple of range queries that should be mutually exclisive

PUT /myindex/_doc/1
{
  "q": {
    "bool": {
      "must": [
        {
          "range": {
            "d": {
              "gt": "now"
            }
          }
        }
      ]
    }
  }
}


PUT /myindex/_doc/2
{
  "q": {
    "bool": {
      "must": [
        {
          "range": {
            "d": {
              "lt": "now"
            }
          }
        }
      ]
    }
  }
}

3. Perform a percolate query sorted by "_doc"

GET /myindex/_search
{
  "query": {
    "percolate": {
      "field": "q",
      "document": {
        "d": "2020-02-18T15:00:00.000+11:00"
      }
    }
  },
  "sort": [
    "_doc"
  ]
}

Expected: Only one of the saved queries should match (document id "2").

Actual: Both of the saved queries match.

Note that you do get the correct behaviour if you do not sort the results. ie.

GET /myindex/_search
{
  "query": {
    "percolate": {
      "field": "q",
      "document": {
        "d": "2020-02-18T15:05:00.000+11:00"
      }
    }
  }
}

[elasticmachine]

Pinging @elastic/es-search (:Search/Percolator)

[matriv]

Reproduces in master as well.

[romseygeek]

This is indeed the same problem as #52617, and is fixed by #52748. I'll open a PR to add specific testing for this scenario.