[GitHub] [accumulo] EdColeman commented on pull request #3375: Fix issue #3371 - tranform Error to RunTimeException in scans

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

EdColeman commented on PR #3375:
URL: https://github.com/apache/accumulo/pull/3375#issuecomment-1536265308

   I picked SourceSwitchingIterator after a brief internal discussion and that was where the issue reported was throwing the exception.  There was not an alternate, obvious iterator that would at least demonstrate the problem.
   
   The only real benefit of this PR was being able to reproduce the stack trace as reported in the issue quickly.  It also identified what possible locations would need protection.
   
   One underlying issue that has been discussed before, but I do not recall a resolution or even a proposed approach is how to isolate "user errors" from system errors and allowing a "user error" to halt otherwise healthy services.  This PR does not come close to solving that, other than this is "what not to do"
   
   I converted this to draft and would just have closed it, but maybe something can come from the discussion?
   


-- 
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