ACCUMULO-3279 Remove version argument from recursiveDelete

`recursiveDelete` is much like a big hammer: get rid of
all of the nodes beneath this parent. As such, it doesn't
make sense to pass in a version, nor does it make sense to
provide a specific version to delete. We can pass in a version
of '-1' which should eliminate the need for any special handling
around BadVersionExceptions.
Merge branch '1.5' into 1.6

Conflicts:
	fate/src/main/java/org/apache/accumulo/fate/zookeeper/IZooReaderWriter.java
	fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooReaderWriter.java
	fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooUtil.java
Merge branch '1.6'
ACCUMULO-3182 coverity found an unused update to a local variable
ACCUMULO-3263 coverity found a strint-to-byte conversion using the default character type
ACCUMULO-3213 Archive files that the garbage collector would otherwise delete.
ACCUMULO-3213 Fix the moveToTrash method name to be more accurate (archive or move to trash)
ACCUMULO-3213 Make DeleteTable archive files if configured.

Archiving a table directory is a little more complicated since we
need to account for the potential existence of files for that table
which were already archived. Ultimately, we're merging the table directory
into the correct directory inside the fileArchive directory.
ACCUMULO-3213 Fix up SimpleGarbageCollectorTest
Merge branch '1.6'
ACCUMULO-3213 Fix up merge issues.