Christopher L. Shannon
Christopher L. Shannon
@EdColeman - Thanks for the quick review, I'll look at making these changes when I get some time later this week either tomorrow or this weekend.
@EdColeman and @dlmarion - I updated my PR with a new commit to address the comments. I renamed the enum to TxInfo and the methods. Also another change is the...
Can someone assign the issue to me? I can take a look at it.
> See[ #2829 ](https://github.com/apache/accumulo/pull/2829) for a discussion on changing the name of prop_debug to something that reflects what is stored, vs how it can be used. Is the intention here...
Yeah, if the consensus is that the name "property" is a bad name about what is stored and something like "node data" is better than might as well just change...
> We have a `checkTablets` command under Admin that may be similar to this utility. I am going to look at it more closely but I think this could go...
@milleruntime - This has been updated to move to the Admin command
@milleruntime - This has been updated to move to the Admin command
> Looks good so far but I think the command could be improved a little bit. The default behavior (if the user doesn't provide a parameter) should just be to...
@milleruntime - Can this issue and also #2814 be merged or do we want to try and get rid of SiteConfiguration now? I think it makes more sense to merge...