Merge pull request #502 from mikewalch/434-exec

Replace Anonymous types with lambdas

 #408 - Removed ClientConfiguration (#500)

#434 - Replace PrivilegedActionException with lambda

 #434 - Replace ClientExec anonymous types with lambdas

fixes #469 made bulk import scan split tolerant (#488)

 * Made MetadataScanner check structure and retry when linked list broken
 * Removed TabletIterator
 * Made code that used TabletIterator use MetadataScanner

Fixes from automated code inspections (#498)

* Simplified JUnit assertions using automated tool

* Fixed javadocs that reference themselves

Make RFileScanner use table props throughout (#493)

Merge branch '1.9'

Use full path as key in file len cache #467

I had used just the file name as the file len cache key, however I realized
file names created by older versions of Accumulo may not be unique.  However,
the path should be unique for older versions because the tablet dir+file name
was unique.

Merge branch 'master' of github.com:apache/accumulo

 #408 - Remove deprecated ClientConfiguration from MapReduce API (#489)

* Removed unnecessary fields in RangeInputSplit
* Updated init method in AbstractInputFormat

Inline inner Rfile classes and interfaces (#487)

* Eliminated CachableBlockFile.Writer by inlining BCFile.Writer
* Made BCFile close its OutputStream
* Eliminated PositionedOuputs wrapping and PositionedOuput interface
* Made RateLimitedOutputStream extend DataOutputStream and take FSDataOutputStream

Add retry mechanism to AccumuloReloadingVFSClassLoader (#453)

 * Also addresses issues mentioned in ACCUMULO-1507

fixes #467 cache file lengths

Commit generated code from Thrift 0.11.0

Update Thrift to latest (0.11.0)

Remove workarounds for bugs fixed in 0.11.0:
* Remove workaround for spammy debug print statement (THRIFT-4062)
* Remove workaround for unused imports in enums (THRIFT-4073)
* Remove RpcWrapper (more details below)

***
RpcWrapper was added in ACCUMULO-2950 to catch server-side
RuntimeExceptions and notify the client with a TApplicationException, as
in Thrift 0.9.0 (See THRIFT-1805). It is no longer needed since a flag
was added to the generator to optionally handle RuntimeExceptions this
way, which we now set in our generate-thrift.sh.

Inline BlockFile interfaces. Fixes #480 (#484)

* Replace BlockRead with CachedBlockRead
* Replace BlockFileReader with CachableBlockFile.Reader
* Replace ABlockReader with CachableBlockFile.CachedBlockRead
* Replace BlockWrite with BCFile.Writer.BlockAppender
* Replace ABlockWriter with BCFile.Writer.BlockAppender
* Replace BlockFileWriter with CachableBlockFile.Writer

Merge branch '1.9'

Fix #448 Remove unused property from test config

Followup from #457; remove unused property from config file

Removing unused properties for #448 (#457)

Work around compilation bug in Eclipse Oxygen

Fixup generics parameter types in generated SimpleThriftService after
generating with `-Pthrift` profile activation.

This works around https://bugs.eclipse.org/bugs/show_bug.cgi?id=534559

Fix formatting for #482

Fixes #414 - Create Connector for new user from existing (#455)

Removed unnecessary interface modifiers (#482)

[maven-release-plugin] prepare for next development iteration

[maven-release-plugin] prepare release rel/1.9.1

Merge branch '1.9'

Update formatter to avoid bad enum formatting

Fix formatting for #481

Add ZStandard compression codec. Fixes #438 (#456)

Cleaned up comments (#481)

Merge pull request #479 from mikewalch/auto-fixes

More intellij automated fixes

Remove redundant 'new' expression in constant array creation

Merge branch '1.9'

Remove warning and fix compilation with Java 7

ACCUMULO-2537 Add @Override Annotations

Added a profile 'use-errorprone' to enforce use of the override annotation on
required methods. Uses google errorprone to handle enforcement. It
ignores the generated code in the project. Findbugs/Spotbugs do not
currently enable enforcement of the annotation, thus the use of
errorprone. Initially setup as a profile to verify it works as expected
and satifies the goal of the annotation enforcement.

Addressed some issues brought up via Github review. Included removal of
some unnecessary tags and the addition of a comment explaining the
profile. Also renamed to 'use-errorprone'.

Removed unnecessary calls to toString() (#478)

Replace explicit types with diamond operator

Remove the fate module by combining it into core (#466)

fix build problems from #458

Merge branch '1.9'

fixes #477 Stop putting -1 in WAL (#458)

fixes #449 code review updates (#458)

Fixes #449 suggested updates to PR apache/accumulo#458

fixes #449 fix two bugs with WAL recovery (#458)

 * Fix bug where tablet is unloaded, reloaded on tserver, and then tserver dies
 * Fix bug with out of order logs.  Recovery code assumed logs were passed in
   time order.  However, since 1.8.0 they have been passed in random order. Rewrote
   recovery code to handle out of order logs.  The fix was to read all logs in
   a sorted merged way.

Fixed more javadoc errors

Fixed javadoc build error

Merge branch 'ACCUMULO-4813-fmt'

ACCUMULO-4813 New bulk import method (continued...)

 * Updated documentation
 * Fixed bug in API
 * Unit tested merge detection code (and found bugs in the proccess)
 * Updated existing bulk IT to use new and old APIs
 * Addressed many TODOs

Fix formatting

Merge pull request #464 from mikewalch/redundant-code

Remove redundant code

Merge branch '1.9'

Removed redundant array creation

Remove empty no args constructors

Remove redundant call to String.format

Removed redundant conditional expression

Remove redundant if statements

Removed redundant local variables (#462)

Remove dead code in bcfile (#460)

fixes #432 Made GC clean up recovery logs (#444)

Fix #461 Handle race condition in AccumuloMonitorAppenderTest

Catch NoSuchElementException and handle it in the
AccumuloMonitorAppenderTest. This can happen if the test sees the state
of the appender while it is being updated from the old monitor location
to the new location.

Updates to the appender occurs in a single thread. Dispatching log
events are synchronized, and thread-safe, so this only ever affected the
test itself.

Fix formatting

 #408 - Removing ClientConfiguration from Proxy code (#451)

* Updated proxy.properties
* Proxy command now takes proxy.properties & accumulo-client.properties
* Removed proxy from accumulo-service

PR Updates

* Update ordering of new thrift objects
* Move Property parsing to ConfigTypeHelper
* Fix Javadoc

Remove deprecated from flush API (#437)

 #408 - Removed more uses of deprecated ClientConfiguration (#450)

* Update checkstyle to ignore line length checks for lines
  with org.apache.accumulo in it

 #408 - Fixed Kerberos ITs (#445)

Merge branch '1.9'

#441 always use hashset for currentlogs

Merge branch '1.9'

fixes #441 only dereference WALs after minc finishes (#443)

* fixes #441 only dereference WALs after minc finishes

Remove unused WAL props (#447)

Expose negation to shell grep command (#446)

When filtering in shell config command, include values (#442)

* Leave current behavior as is, but add new option to search values

Merge branch '1.9'

Fix #440 Add SHA512 generation to vote email

* Also remove unnecessary comments
* Make general improvements so script can be reused for other projects
* Make other improvements to avoid shellcheck-identified problems

 #408 - Removed uses of deprecated ClientConfiguration (#430)

Merge branch '1.9'

Fix #433 Update copyright dates to 2018

Fix formatting

ACCUMULO-4813 Add permission checks to client (continued...)

* Fix bug for single tablet ingest
* Fix BulkImportMove
* Created 2 new tests in BulkLoadIT
* Catch ExecutionException in Move

Merge pull request #435 from mikewalch/acc-434

Replace anonymous types with lambdas

 #434 - Replace anonymous FileFilter with lambda

 #434 - Replace anonymous WaitCondition with lambda

 #434 - Replace anonymous Callable types with lambda

Merge branch '1.9'

Fix formatting

Handle RW unexpected table state (#429)

 #434 - Replaced Comparator anonymous types with lambdas

 #434 - Replaced IAnswer anonymous types with lambda

Made all long literals end with 'L' (#431)

Merge branch '1.9'

 #408 - Removed ClientConfiguration from ClientOpts (#419)

Separate mapreduce code from core (#425)

* Move mapreduce client code to separate module
* Ensure "fresh start" by cleaning out old (deprecated) APIs

Reverse sort order except for state and table name (#426)

[maven-release-plugin] prepare for next development iteration

[maven-release-plugin] prepare release rel/1.9.0

ACCUMULO-4697 tserver could start too many idle compactions

Tablet.initiateMajorCompaction() always returns false, but we check the
return value when accounting for the number of idle major compactions
started in TabletServer.MajorCompactor.run() and continue to kick off
idle compactions on every tablet.

This ticket removed the idle compaction code and modified
initiateMajorCompaction() to return void rather than a constant false
value.

Fix broken ConnectorImpl

Found and fixed broken switch cases (fall-through) in ConnectorImpl,
based on (non-deprecation related) compiler warnings.

Also fixed another warning about unused variables from a MapReduce IT.

ACCUMULO-4813 New bulk import method (continued...)

 * In load RPC aow sending multiple file per tablet instead of one
 * Now saving mapping file to HDFS on client side
 * Simplified JSON serialization
 * Refactored new and old bulk import code into two packages
 * Now only iterate over mapping in master, no longer load into memory