[GitHub] [accumulo] keith-turner commented on issue #1415: Update Apilyzer to treat core.iterators as SPI

[GitBox <gi...@apache.org>]

keith-turner commented on issue #1415: Update Apilyzer to treat core.iterators as SPI
URL: https://github.com/apache/accumulo/issues/1415#issuecomment-553463769
 
 
   >  Perhaps SPI should be declared as public API along with iterators.
   
   Pluggable interfaces in SPI related to scan execution and caching are tightly coupled to Accumulo's internal model for executing scans.  If these interfaces are in the API it could make it very difficult to change the internal execution model for scans.  The reason is because it may require going through a deprecation cycle where a single release supports an old and new model of executing scans, which may not be practical.  Also the interfaces related to scan execution and caching only impact scan performance, never scan data returned.  

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services