This reverts commit 495929dc.

parent.relativePath was being set to the default value unnecessarily.

* changed direct access of timestamp field to use getTimstamp() instead inside of calculateElapsedTime.

* Added support to the tomcat Server adaptor for correctly getting request start time. (Includes additional dependency on tomcat-coyote)
* Update layouts.html to document the addition of elapsedTime/D to the pattern support for IAccessEvent
* Modified AccessEvent.java to calculate the elapsedTime at construction of the event, instead of repeatedly in the getter.  This allows the value to be more easily seen in a debugger, and cleans up the code slightly.

* changed direct access of timestamp field to use getTimstamp() instead inside of calculateElapsedTime.

* Added support to the tomcat Server adaptor for correctly getting request start time. (Includes additional dependency on tomcat-coyote)
* Update layouts.html to document the addition of elapsedTime/D to the pattern support for IAccessEvent
* Modified AccessEvent.java to calculate the elapsedTime at construction of the event, instead of repeatedly in the getter.  This allows the value to be more easily seen in a debugger, and cleans up the code slightly.

ExecutorService.shutdownNow returns the list of jobs that never
executed, not the list of jobs that didn't terminate.

Mutable fields in SocketAppenderBase were previously exposed as
protected fields and modified by constructors in concrete subclasses.
This commit improves encapsulation of the base class by exposing
protected constructors for subclasses to invoke rather than directly
exposing the fields.  These new constructors are marked as deprecated,
as the convenience constructors on the base classes (which are also
deprecated) are the sole reason for their existence.  A future release
should remove all but the no-arg constructor from the base class and
subclasses.

These test classes were really testing SocketAppenderBase not
SocketAppender, per se.  The new implementation of SocketAppenderBase is
significantly different and will require a new unit test suite.

Previously, the executor service used here was not consistent with a
recent change to ContextBase (to allow more than one concurrent
asynchronous task).  This commit also ensures that the executor service
is shut down when the valve is stopped.