Fix match query on a scaled_float property no longer matches for some values

[kkewwei]

Description

This PR only addresses the mismatch issue.

As it's known, float and double types suffer from precision loss, and it's inevitable. When reading doc_value in the filed(such as range, sorting, or aggregation), it can lead to unforeseen exceptions. The root cause lies in the fact that the precision of float/double data is already lost before storing.

OpenSearch/modules/mapper-extras/src/main/java/org/opensearch/index/mapper/ScaledFloatFieldMapper.java

Line 468 in 9db5e67

long scaledValue = Math.round(doubleValue * scalingFactor);

It can be extremely terrifying when related to money.

To solve it, I will submit a new PR. In this PR, I will create a new type big_decimal, supported by BigDecimal to effectively eliminate the precision loss.

Related Issues

Resolves #12433

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 40380fb: 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 adbf317: 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 66bb50e: null

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?

[kkewwei]

❌ Gradle check result for 66bb50e: null

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?

MinimumClusterManagerNodesIT.testThreeNodesNoClusterManagerBlock #14289
RemoteStoreBackpressureAndResiliencyIT.testWritesRejectedDueToTimeLagBreach #12411
DedicatedClusterSnapshotRestoreIT.testSnapshotWithStuckNode #15806

[kkewwei]

@bowenlan-amzn would you mind have a look in your spare time, very thank you.

[github-actions]

❌ Gradle check result for 75cea4e: 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 88bcdda: 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?

[kkewwei]

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

EhCacheDiskCacheTests.testComputeIfAbsentConcurrently #16159

[github-actions]

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

[kkewwei]

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

RemoteStorePinnedTimestampsGarbageCollectionIT.testLiveIndexNoPinnedTimestampsWithExtraGenSetting #16088
DedicatedClusterSnapshotRestoreIT.testSnapshotWithStuckNode #15806

[opensearch-trigger-bot]

This PR is stalled because it has been open for 30 days with no activity.

[kkewwei]

Closing in favor of #19188.