cshannon commented on PR #3001: URL: https://github.com/apache/accumulo/pull/3001#issuecomment-1281495012
> @cshannon I opted to just replace the TODO comments and open up #3027 in their place. Let me know if you think this is fine or if I should remove the whole if statements and comments all together. I'm ok either leaving the comment and if statement to fill in later or just removing it as it's just a test class. The new updated comment here is still basically a TODO (without calling it TODO) so not sure it's any better than before. To me the best way to track it is the new issue which you just created so I'm not sure we need to leave it and maybe can just remove the entire empty if statement block. @ctubbsii - any opinion on this? -- 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]
