Skip to content

Consolidate iterator parsing code#6111

Open
DomGarguilo wants to merge 1 commit intoapache:mainfrom
DomGarguilo:iteratorCheckCentralize
Open

Consolidate iterator parsing code#6111
DomGarguilo wants to merge 1 commit intoapache:mainfrom
DomGarguilo:iteratorCheckCentralize

Conversation

@DomGarguilo
Copy link
Member

Fixes #6074

Create new IteratorSettingUtil to consolidate iterator parsing/validation code from table and namespace operations into one place.

This new IteratorSettingUtil consolidates and improves the various checks and validation that were being done in both of these places.

@DomGarguilo DomGarguilo added this to the 4.0.0 milestone Feb 4, 2026
@DomGarguilo DomGarguilo self-assigned this Feb 4, 2026
@DomGarguilo
Copy link
Member Author

Any changes made should avoid impacting scan performance. Scans need to efficiently parse iterator config. For other code parsing iterator configuration, efficiency is less of a concern.

This is from #6074. This PR does not alter the iterator parsing for scans.

@keith-turner
Copy link
Contributor

In #6040 there is a new class IteratorProperty.java that parses and checks a single iterator prop key/value. That could eventually be used in these changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Iterator parsing code is duplicated

2 participants