LIB: Add StartupException and subclasses.

This change adds StartupException, and subclasses InvalidConfigurationException,
and UnsupportedPlatformException. StartupExeceptions can be thrown by the
Adaptor.init() method to immediatly abort Application startup.
StartupExceptions bypass the Application retry with back-off wait recovery
mechanism that will re-attempt to start the adaptor in hopes that a recoverable
error has been corrected. Therefore StartupExceptions should be thrown for
unrecoverable situations, such as running on the wrong platform or a
configuration that could never be valid.

[pjo, 2014-05-01 00:57:58]

Thank you.

Please make sure to run the code (say with
`ant run' and modified AdaptorTemplate to throw
after some delay) to make sure that the program
actually does really exit.

My concern is that only one thread is killed 
instead of the program really exiting.

Thank you,
PJ

https://codereview.appspot.com/99980043/diff/1/src/com/google/enterprise/adap...
File src/com/google/enterprise/adaptor/InvalidConfigurationException.java
(right):

https://codereview.appspot.com/99980043/diff/1/src/com/google/enterprise/adap...
src/com/google/enterprise/adaptor/InvalidConfigurationException.java:32: *
Constructs a InvalidConfigurationException with a supplied message but no root
over80

https://codereview.appspot.com/99980043/diff/1/src/com/google/enterprise/adap...
File src/com/google/enterprise/adaptor/StartupException.java (right):

https://codereview.appspot.com/99980043/diff/1/src/com/google/enterprise/adap...
src/com/google/enterprise/adaptor/StartupException.java:81: }
Not sure if it's a good idea to chain these.
Won't logging show the various messages anyway?

[Brett, 2014-05-01 03:47:51]

>Please make sure to run the code (say with
>`ant run' and modified AdaptorTemplate to throw
>after some delay) to make sure that the program
>actually does really exit.

>My concern is that only one thread is killed
>instead of the program really exiting.

You are correct.  That is exactly what happens.

https://codereview.appspot.com/99980043/diff/1/src/com/google/enterprise/adap...
File src/com/google/enterprise/adaptor/InvalidConfigurationException.java
(right):

https://codereview.appspot.com/99980043/diff/1/src/com/google/enterprise/adap...
src/com/google/enterprise/adaptor/InvalidConfigurationException.java:32: *
Constructs a InvalidConfigurationException with a supplied message but no root
On 2014/05/01 00:57:58, pjo wrote:
> over80

Done.

https://codereview.appspot.com/99980043/diff/1/src/com/google/enterprise/adap...
File src/com/google/enterprise/adaptor/StartupException.java (right):

https://codereview.appspot.com/99980043/diff/1/src/com/google/enterprise/adap...
src/com/google/enterprise/adaptor/StartupException.java:81: }
On 2014/05/01 00:57:58, pjo wrote:
> Not sure if it's a good idea to chain these.
> Won't logging show the various messages anyway?

This was just straight-up copied from the Connector Manager's
exceptions.  If printing stack traces, then you are correct.

[Brett, 2014-05-01 05:00:48]

Add call to Application.stop() in case of StartupException.  I strongly suspect
IOException should do the same.

[Brett, 2014-05-02 00:18:34]

Call Application.stop() for all abnormal terminations.

[pjo, 2014-05-02 01:35:54]

FYI: I'll prolly not look at this CL before Monday. Thank you.
Can you also remove the special getMessage handling?


- technology's compounding interest
-


On Thu, May 1, 2014 at 5:18 PM, <Brett.Michael.Johnson@gmail.com> wrote:

> Call Application.stop() for all abnormal terminations.
>
> http://codereview.appspot.com/99980043/
>

[Brett, 2014-05-02 02:43:43]

Remove override of Exception.getMessage()

[brett.michael.johnson_gmail.com, 2014-05-02 02:44:47]

On May 1, 2014, at 6:35 PM, PJ <pjo@google.com> wrote:

> FYI: I'll prolly not look at this CL before Monday. Thank you.

Darn. I was hoping to get this in for tomorrow’s Vizdom iteration.

> Can you also remove the special getMessage handling?

Done.

> 
> 
> - technology's compounding interest
> -
> 
> 
> On Thu, May 1, 2014 at 5:18 PM, <Brett.Michael.Johnson@gmail.com> wrote:
> Call Application.stop() for all abnormal terminations.
> 
> http://codereview.appspot.com/99980043/
> 

[pjo, 2014-05-02 02:49:03]

1 other quick comment.

Thank you

https://codereview.appspot.com/99980043/diff/60001/src/com/google/enterprise/...
File src/com/google/enterprise/adaptor/InvalidConfigurationException.java
(right):

https://codereview.appspot.com/99980043/diff/60001/src/com/google/enterprise/...
src/com/google/enterprise/adaptor/InvalidConfigurationException.java:61:
super(null, rootCause);
remove comment?

[Brett, 2014-05-02 02:57:40]

Remove override of Exception.getMessage()

[Brett, 2014-05-02 02:59:23]

https://codereview.appspot.com/99980043/diff/60001/src/com/google/enterprise/...
File src/com/google/enterprise/adaptor/InvalidConfigurationException.java
(right):

https://codereview.appspot.com/99980043/diff/60001/src/com/google/enterprise/...
src/com/google/enterprise/adaptor/InvalidConfigurationException.java:61:
super(null, rootCause);
On 2014/05/02 02:49:03, pjo wrote:
> remove comment?

I removed the method, since it depended on the behavior of the overrides
getMessage().

[pjo, 2014-05-06 00:43:31]

LGTM.  Thank you

[pjo, 2014-05-07 00:14:01]

Please submit.

Thank you,
PJ


- technology's compounding interest
-


On Mon, May 5, 2014 at 5:43 PM, <pjo@google.com> wrote:

> LGTM.  Thank you
>
> https://codereview.appspot.com/99980043/
>

[Brett, 2014-05-07 00:33:43]

Committed 06 May 2014 to Adaptor Library:
To https://code.google.com/p/plexi/
   8b19c07..97c617d  master -> master