Clean up BaseDBConfiguration class
authorRyan Goulding <ryandgoulding@gmail.com>
Wed, 13 Sep 2017 20:58:19 +0000 (16:58 -0400)
committerRyan Goulding <ryandgoulding@gmail.com>
Wed, 13 Sep 2017 21:04:55 +0000 (17:04 -0400)
commit222513c30f3c0c0e7bfce48b6c96d9ddfba75dba
tree87fa90d2c185337f1da46e793d3eefe06e69bf04
parentd315e6996052dc81c7e5deebd7fc4229a4e93c0c
Clean up BaseDBConfiguration class

This change isn't too big;  it mainly just attempts to clean up the
BaseDBConfiguration class using best practices.  For example:

* documentation is added surrounding public methods, public constants,
  protected variables, and class headers.
* code logic for parsing "optional" integer properties is abstracted
  into a separate private static function, which returns a default
  value if the given input property cannot be parsed or is not present.
* public functions where the NumberFormatException RuntimeException
  might be thrown have the appriorate "throws" modifier added to their
  function signatures to indicate that the consumer is responsible for
  exception handling.
* constants are extracted in place of "magic numbers"

This is really just code sanitation work.

Change-Id: Ifaf092e10b17f54c7cce0a888aa49bfe8377cdd3
Issue-Id: SDNC-79
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
dblib/provider/src/main/java/org/onap/ccsdk/sli/core/dblib/config/BaseDBConfiguration.java