keith-turner commented on pull request #2096: URL: https://github.com/apache/accumulo/pull/2096#issuecomment-845161781
> These changes help, thanks. I am still wondering why we need both constructors? If you make the jobSelectedAll an Optional type parameter wouldn't that suffice? @milleruntime I made the constructor take an optional and added a comment about the parameter in 28c84b6 -- 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: [email protected]
