support rangeQuery and regexpQuery in constant_keyword field type

[kkewwei]

Description

Support rangeQuery and regexpQuery in constant_keyword field type

Related Issues

Resolves #[Issue number to be closed when this PR is merged]

Check List

• Functionality includes testing.
• API changes companion pull request created, if applicable.
• Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

[github-actions]

❌ Gradle check result for 72a3ba8: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[github-actions]

✅ Gradle check result for 3cd9c3e: SUCCESS

[codecov]

Codecov Report

Attention: Patch coverage is 88.23529% with 2 lines in your changes missing coverage. Please review.

Project coverage is 71.86%. Comparing base (d4e7766) to head (0ebfcb3).

Files	Patch %	Lines
...earch/index/mapper/ConstantKeywordFieldMapper.java	88.23%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main   #14711      +/-   ##
============================================
- Coverage     71.87%   71.86%   -0.01%     
- Complexity    62811    62824      +13     
============================================
  Files          5169     5169              
  Lines        294646   294663      +17     
  Branches      42610    42616       +6     
============================================
- Hits         211778   211773       -5     
- Misses        65415    65447      +32     
+ Partials      17453    17443      -10     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[kkewwei]

@gaobinlon, can you please have a look when you are free, thank you.

[kkewwei]

@dblock can you help review the PR? very thank you.

[dblock]

Looks good. Definitely needs a CHANGELOG.

Please don't forget to add spec and tests to https://github.com/opensearch-project/opensearch-api-specification.

[github-actions]

❌ Gradle check result for 50459c2: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[github-actions]

❌ Gradle check result for e72dae3: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[github-actions]

❌ Gradle check result for abb776f: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[github-actions]

✅ Gradle check result for 0ebfcb3: SUCCESS

[reta]

@msfroh your concerns cleared out? thanks!

[msfroh]

For IndexFieldType, it is used for index name matching. Index name matching can support wildcard matching, but does not support regexp and range matching.

Not yet!

So it seems useless to putrangeQuery and regexpQuery in the parent class at present .

The idea is that if we do move it to the parent, then we can support these query types on index names.

Again, though -- I think we can safely defer that to another issue. (I don't even know if anyone actually wants to be able to filter on index names by range or regex.)