Ignore attribute awareness when custom preference is set

[finnegancarroll]

Description

This change addresses a bug which appears when a custom preference string is included with a query while awareness attributes are present on the cluster.

In this scenario it can seem as if the custom preference string included with query is not being honored. This is due to how awareness attributes preference shards which share an attribute with the coordinating node when routing a search request. Awareness attributes take priority and the custom string preference is only applied after this step. A search request with the same custom string preference can then be routed differently depending on which node is acting as coordinator.

All other preference types exit early and ignore awareness attributes with custom string preference being the one exception that attempts to deterministically route requests within awareness attribute groups.

This change ignores awareness attributes when a custom string preference is present.

Related Issues

Resolves #18817

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 7b2b6f9: 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?

[finnegancarroll]

Flaky - #18850

[github-actions]

✅ Gradle check result for 7b2b6f9: SUCCESS

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.78%. Comparing base (3b7603e) to head (eabfc36).
⚠️ Report is 14 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff              @@
##               main   #18848      +/-   ##
============================================
+ Coverage     72.67%   72.78%   +0.10%     
- Complexity    68610    68612       +2     
============================================
  Files          5577     5577              
  Lines        315375   315374       -1     
  Branches      45772    45773       +1     
============================================
+ Hits         229209   229539     +330     
+ Misses        67613    67154     -459     
- Partials      18553    18681     +128     

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[github-actions]

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

[finnegancarroll]

Flaky test #18872

[github-actions]

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

[finnegancarroll]

Same flaky test #18872.

[github-actions]

✅ Gradle check result for eabfc36: SUCCESS