Wildcard field use only 3-gram to index

[HUSTERGS]

Description

Accorrding the test result of #17099 , change wildcard field to only index 3-gram of the input data, instead of 1-3 gram, to reduce the overall index size caused by long positing list of single and double charaters, which is not really helpful when search. And also help to increase the write speed.

Related Issues

Resolves #17099

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 3abbea7: 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?

[andrross]

@msfroh Is this worth creating a rolling upgrade test case for to ensure compatibility across the versions?

@HUSTERGS Can you rebase this with the latest from main? I think the bwc tests are failing due to a version bump.

[github-actions]

❌ Gradle check result for b3576d9: 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?

[HUSTERGS]

Flaky tests: #17347

[msfroh]

Flaky tests: #17347

Oh, shoot! That one is my fault. I made it "less flaky" a couple of weeks ago, but apparently missed something.

[github-actions]

✅ Gradle check result for b3576d9: SUCCESS

[codecov]

Codecov Report

Attention: Patch coverage is 97.05882% with 1 line in your changes missing coverage. Please review.

Project coverage is 72.45%. Comparing base (38e4b33) to head (5e986b3).
Report is 3 commits behind head on main.

Files with missing lines	Patch %	Lines
...g/opensearch/index/mapper/WildcardFieldMapper.java	97.05%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main   #17349      +/-   ##
============================================
+ Coverage     72.41%   72.45%   +0.04%     
+ Complexity    65569    65557      -12     
============================================
  Files          5291     5291              
  Lines        304338   304321      -17     
  Branches      44181    44179       -2     
============================================
+ Hits         220376   220495     +119     
+ Misses        65917    65751     -166     
- Partials      18045    18075      +30     

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

[andrross]

Thanks @HUSTERGS. Want to take a crack at adding rolling upgrade tests to ensure compatibility across versions? It should be fairly straightforward to add a new case in the rest-api-spec test framework. In the "old_cluster" step you'll write some data to an index and verify a wildcard search returns expected results, then in the "mixed_cluster" and "upgraded_cluster" you'll just do the same search and verify you get the same results. Here's a commit where something similar was done for a geo field: c40014f

[github-actions]

❌ Gradle check result for e972562: 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 d0b29ad: 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 f82a1eb: 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 273ff76: 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?

[HUSTERGS]

There are some test cases added in #16827, which fixed null pointer exception cause by plain text regex query on wildcard field, but is not yet backported to 2.x, the corresponding pr is stalled #17001, so I just removed those test case, so the rolling upgrade tests can run properly

[github-actions]

✅ Gradle check result for 5e986b3: SUCCESS