[I] Iterator parsing code is duplicated [accumulo]

["keith-turner (via GitHub)" <gi...@apache.org>]

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: notifications-unsubscribe@accumulo.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] Iterator parsing code is duplicated [accumulo]

["DomGarguilo (via GitHub)" <gi...@apache.org>]

DomGarguilo closed issue #6074: Iterator parsing code is duplicated
URL: https://github.com/apache/accumulo/issues/6074


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org