keith-turner opened a new issue, #6074:
URL: https://github.com/apache/accumulo/issues/6074
The code for parsing iterator configuration from properties is duplicated
throughout the codebase. The parsing is not consistent in its error checking.
All of this code could be refactored in the following way.
* Create a single method that parses from properties to
`List<IteratorSetting>` (could potentially use existing code).
* Modify any code that is currently doing parsing to use this new method.
The centralized parsing code could validate the following.
* A valid scope is used.
* The value has only two fields seperated by a comma.
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.
The following comment suggested a possible way to start this.
_Originally posted by @keith-turner in
https://github.com/apache/accumulo/pull/6040#discussion_r2714081390_
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
