RCORE-2221 fix RQL BETWEEN regression for properties over links

[ironage]

Introduced in #7785. The between node only handles a single column, not a link chain, so we should check this before using the optimized query node.

Replaces #7940
Fixes #7935

☑️ ToDos

• 📝 Changelog update
• 🚦 Tests (or not relevant)
• C-API, if public C++ API changed
• bindgen/spec.yml, if public C++ API changed

[coveralls-official]

Pull Request Test Coverage Report for Build james.stone_584

Details

• 30 of 30 (100.0%) changed or added relevant lines in 2 files are covered.
• 37 unchanged lines in 13 files lost coverage.
• Overall coverage increased (+0.03%) to 91.113%

---

Files with Coverage Reduction	New Missed Lines	%
src/realm/array_backlink.cpp	1	91.38%
src/realm/dictionary.cpp	1	85.16%
src/realm/sort_descriptor.cpp	1	94.06%
src/realm/sync/noinst/server/server.cpp	1	74.02%
test/fuzz_tester.hpp	1	57.73%
test/test_dictionary.cpp	1	99.83%
test/fuzz_group.cpp	2	54.86%
test/object-store/sync/flx_sync.cpp	2	98.35%
src/realm/sync/client.cpp	3	90.96%
test/test_thread.cpp	3	66.14%

Totals
Change from base Build 2548:	0.03%
Covered Lines:	217072
Relevant Lines:	238246

---

💛 - Coveralls

[ironage]

CI failure was a system issue "[2024/08/07 12:05:22.852] error: unexpected service error: The Xcode build system has crashed. Build again to continue."

[nicola-cab]

@jedelbo @ironage I suspect that range queries over links will be quite slow in case the array is compressed. I'll add a benchmark to verify this.

[ironage]

@nicola-cab you are welcome to make a benchmark. The specialized BetweenNode won't work because the links could point to any array index at random. Not sure there will be any easy way to optimize this case.