handle unsigned long in flush and merge operations for star tree

[Shailesh-Kumar-Singh]

Description

This change handles the comparison and sorting of unsigned long values during flush operations. Since unsigned long values are stored as Java Long (and values greater than Long.MAX_VALUE turn negative),Long.compareUnsigned is used to ensure correct sorting and comparison of unsigned long dimensions. This ensures that unsigned long values are properly handled and sorted during flush operations

Related Issues

Issue - Handle unsigned long during flush and merge operations

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 80db308: 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 f3fb106: 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 6272c9b: 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 be5c59d: 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 f44db3b: 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 87742cc: 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 96d20dc: 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 fa447c8: 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 1394c39: 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 25ab688: 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 3cd0858: 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 147d817: SUCCESS

[codecov]

Codecov Report

Attention: Patch coverage is 89.69072% with 10 lines in your changes missing coverage. Please review.

Project coverage is 72.69%. Comparing base (e6fc600) to head (147d817).
Report is 3 commits behind head on main.

Files with missing lines	Patch %	Lines
...be/startree/fileformats/meta/StarTreeMetadata.java	62.50%	4 Missing and 2 partials ⚠️
...h/index/compositeindex/datacube/ReadDimension.java	75.00%	2 Missing ⚠️
...rtree/fileformats/meta/StarTreeMetadataWriter.java	85.71%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main   #16646      +/-   ##
============================================
+ Coverage     72.34%   72.69%   +0.35%     
- Complexity    65481    65874     +393     
============================================
  Files          5300     5305       +5     
  Lines        304330   304578     +248     
  Branches      44141    44177      +36     
============================================
+ Hits         220158   221421    +1263     
+ Misses        66093    65276     -817     
+ Partials      18079    17881     -198     

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

[Shailesh-Kumar-Singh]

Had to move this to a separate PR
because of a github bug - It was showing 5 random unresolved conversation (which cannot be resolved) and I cannot merge code without resolving all conversations.