refactor: rename RowIdMask to RowAddrMask

[yanghua]

sub-task of #5326

[codecov]

Codecov Report

❌ Patch coverage is 91.25000% with 14 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
rust/lance-index/src/vector/flat/index.rs	55.55%	4 Missing ⚠️
rust/lance/src/io/exec/filtered_read.rs	50.00%	2 Missing and 2 partials ⚠️
rust/lance-index/src/scalar/expression.rs	91.66%	1 Missing and 1 partial ⚠️
rust/lance/src/index/vector/pq.rs	0.00%	2 Missing ⚠️
rust/lance-core/src/utils/mask/nullable.rs	88.88%	0 Missing and 1 partial ⚠️
rust/lance/src/io/exec/utils.rs	66.66%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!

[yanghua]

This work is a subtask of this issue. #5326

[yanghua]

@jackye1995, please take a look, thanks.

[yanghua]

Hi @jackye1995 @wjones127 Please take a look, when you have time.

[majin1102]

Seems a light name changing from row_id to row_addr. Looks good to me

[github-actions]

Review Summary

This is a clean mechanical rename refactoring from RowIdMask to RowAddrMask (and related types like NullableRowIdMask → NullableRowAddrMask, iter_ids → iter_addrs). The changes are part of the larger effort in #5326 to distinguish between row address and stable row ID semantics.

Assessment: ✅ LGTM

No P0/P1 issues found.

The refactoring is straightforward and consistent:

• All type renames are applied uniformly across the codebase
• Variable names are updated to match (e.g., row_id → row_addr where appropriate)
• Comments and documentation updated to reflect the new terminology
• Test names updated (test_row_id_mask → test_row_addr_mask)
• No behavioral changes - pure rename refactoring

The coverage gaps reported by Codecov are on pre-existing uncovered lines, not regressions introduced by this PR.

[jackye1995]

looks good to me!