Disable query rewriter as a default behaviour

[rishabhmaurya]

Description

Disable query rewriter(#19060) as a default behaviour. There have been several issues reported so far. Listing them here soon

Related Issues

• [BUG] Regression (3.2 -> main): Double negation is just negation #19266
• [BUG] must_not that wraps terms query #19566
• [AUTOCUT] Integration Test Failed for observabilityDashboards-3.3.0 dashboards-observability#2497 (no details on the issue yet, but information was shared offline that showed the test failure goes away if this setting is disabled)

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.

[getsaurabh02]

Thanks @rishabhmaurya . We should highlight the related/reported issues here for tracking.

Does this also need any documentation change as it's a change in settings default behaviour?

[andrross]

We should highlight the related/reported issues here for tracking.

@getsaurabh02 @rishabhmaurya I added the 3 issues I'm aware of to the description

[jainankitk]

@peterzhuamazon - Do we need the backport label since we have the manual backport PR already open?

[github-actions]

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

[peterzhuamazon]

@peterzhuamazon - Do we need the backport label since we have the manual backport PR already open?

Nope, I didnt realize there was a manual backport, see Rishabh already remove the label now 👍

[github-actions]

✅ Gradle check result for a010db9: SUCCESS

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.97%. Comparing base (8a1c55f) to head (a010db9).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff              @@
##               main   #19592      +/-   ##
============================================
- Coverage     73.07%   72.97%   -0.11%     
+ Complexity    70635    70569      -66     
============================================
  Files          5723     5723              
  Lines        323490   323490              
  Branches      46851    46851              
============================================
- Hits         236401   236054     -347     
- Misses        67977    68469     +492     
+ Partials      19112    18967     -145     

☔ 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.