milleruntime commented on pull request #2248: URL: https://github.com/apache/accumulo/pull/2248#issuecomment-910213010
> Changes made in #2252 seem to have fixed the issue in #2239. I still think some of the changes here are useful to keep, specifically moving the sanity check out of the files constructor and reducing to only one copy of runningJobs. I agree there is some usefulness to changes here. I do feel like we should leave the exceptions in the constructor though since they served their original purpose of getting our attention. I don't want a future bug to go unnoticed in the debug logging. I will redo this PR. -- 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]
