[mattisonchao]

Part of oxia-db/oxia#1759. Server side: oxia-db/oxia#1798.

Problem

list() is deprecated in favor of rangeScan() (#485), but a range scan returns the values too, so moving a caller from list() to rangeScan() transfers every value.

Change

• RangeScanOption.IncludeValue(boolean), which returns the OptionIncludeValue of GetOption.IncludeValue. With false, the client sets include_values = false on the request, and the results have an empty value and the version of each record. With an index, they still have their secondary keys.
• include_values is an optional bool in client.proto: unset means true, so the requests of older clients keep their values.
• An older server ignores the field and sends the values, so the client drops them. No feature flag is needed.
• The deprecation of list now points to rangeScan with IncludeValue(false).

Testing

• AsyncOxiaClientImplTest.rangeScanWithoutValuesDropsTheValuesOfAnOlderServer: the request sets include_values = false only without the values, and the client drops the values a server sends anyway.
• OptionsUtilsTest.includesValue: the values are included unless an option leaves them out.
• OxiaClientIT.testRangeScanWithoutValues: against a server, the scan returns the keys and the versions with empty values.
• ./gradlew spotlessApply build, including the integration tests: pass. check-coverage.py: 100% of lines (3636) and branches (1534).

AI Usage

Claude Code made the change, wrote the tests, ran the build, wrote this description and created the PR, at the contributor's request.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!