milleruntime commented on pull request #2256:
URL: https://github.com/apache/accumulo/pull/2256#issuecomment-910487770


   > This change does make some of the methods shorter, but IMO it does not 
make them easier to understand vs the switch stmts.
   
   The goal was to make it easier to understand but I only have a limited 
understanding of the code. This was only one attempt an a more object oriented 
approach. Any ideas of how this could be more Object oriented?


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


Reply via email to