Christopher L. Shannon

Results 148 comments of Christopher L. Shannon

@jbonofre - I am minus -1 on the merge for now as I am not sure if there is a consensus. @gtully was looking at this quite a bit when...

@milleruntime - Any thoughts on this before merging since you looked at the others?

> > Found another utility that can be moved to the accumulo admin command. This utility cleans all old instances out of Zookeeper (except the current one). It's a bit...

Closing this PR out as I'm going to create a new one and just merge the functionality of CleanZookeeper into the DeleteZooInstance utility with a flag as suggested.

@EdColeman - Here is the version version of the new command. There could be some more enhancements done either on this PR or a follow on. Some other stuff we...

My vote would be to make this new command the new `du` command and just remove the old one entirely if the outputs are the same. The only reason to...

> The file sizes stored in the metadata table are estimates. I see 2 options: > > 1. Create a new option to the `du` command to call the new...

I think I like the idea of just dropping the old command entirely and having people just use the hadoop command directly if they really want the hdfs size. The...

@ctubbsii - Thanks for the detailed feedback, that's very helpful to get some more background. My comments are below to address each of your points. 1. I agree that we...

@keith-turner - Thanks so much for that detailed explanation and example of how things could be an estimate. I had mostly been looking at bulk imports and hadn't gotten as...