[master] Update classification document to match the debug output

The debug output was modified to output text strings using
single quotes instead of double quotes but the document missed
getting this update.

[master] Add commit number to change log for 4480

[master] Merge branch 'trac4480'

Add debug statements to the classification code

[master] Fixed Ubuntu build complaint

Added <stdint.h> to src/lib/dhcpsrv/pgsql_exchange.h

[4480] Fixed a few typos

    Minor corrections to doc/guide/classify.xml

[master] Created common PgSqlConnection class from PgSqlLeaseMgr

    Merged in trac 4276.

[trac4480] Update per review comments

Create a LogContentTest class and move the functions to manipulate
a test to check on the log output there.  This will make it available
for other tests in the future.  As part of this bundle a bit more
work into the class to minimze the calls in the test routines.

Where reasonable remove extra copys of the values being pushed
onto the stack - instead of generating a copy just for the logging
simply use the value from the top of the stack directly.

Swtich to using single quotes around text values.

Added some more description in the classify document

[4276] Addressed review comments #2

Moved PgSqlExchange unit tests into their own file.
Improved time testing.

[4276] Addressed review comments

Minor cleanups, added some unit testing of PgSqlExchange functions.

[4480] Add debug logging for evaluation of classification expressions

This patch adds a set of log messages to indicate what values are
being popped from or pushed to the classification value stack.  This
is meant to be used by an administrator when debugging the expression
statements.

[master] ChangeLog for trac3164

[master] Merge branch 'trac3164'

[3164] Add changes suggested by review

Change the connection timeout parameter from an "int" to an "unsigned
int". Update the checks to allow for lexical_cast not throwing an
exception when converting a string representing a negative number
to an unsigned int.

[master] Added ChangeLog entry for #4495.

[master] Merge branch 'trac4495'

[master] Added ChangeLog entry for #4499.

[master] Merge branch 'trac4499'

[4276] Replace use of PGresult with PgSqlResult throughout

[4276] Created new base class, PgSqlExchange

src/lib/dhcpsrv/Makefile.am
    Added pgsql_exchange.cc and pgsql_exchange.h

src/lib/dhcpsrv/pgsql_exchange.h
src/lib/dhcpsrv/pgsql_exchange.cc
    New files, containng new base class PgSqlExchange from
    which was distilled from PgSqlLeaseExchange

src/lib/dhcpsrv/pgsql_lease_mgr.cc
    Refactored exchange classes to use new base class
    Moved PsqlBindArray into pgsql_exchange.*

[4499] Removed inefficient conversion of the host identifier to text.

[4276] Initial impl of PgSqlConnection class

    Initial refactoring of Postgresql connection logic out of
    PgSqlLeaseMgr into new PgSqlConnection.

[4495] Fixed inefficient creation of DHCP Message Type option.

[master] Added ChangeLog entry 1115 for #4481

[master] Added query<x> arugment to several callouts

    Merge branch 'trac4481'

[4481] Changed lease6_select API doc to show query6 as "in" only

[3164] Updated kea-admin copyright date

[4481] Add missing description of query6 argument to lease6_select

[3164] Correct a typo in the user guide

[master] Added ChangeLog entry for #4485

[master]Merge branch 'trac4485'

[3164] The database connection timeout is now a configurable parameter

In addition, the default has been changed to five seconds.

[4481] Made query4 and query6 callout arguments uniformly available

The client packet is now uniformly available to all client packet driven
callouts for both v4 and v6:

Added "query4" to lease4_select and lease4_renew
    src/bin/dhcp4/dhcp4_hooks.dox
        Added query4 argument to lease4_select and lease4_renew documentation

    src/bin/dhcp4/tests/hooks_unittest.cc
        Revamped to track both query4 and response4 arguments
        TEST_F(HooksDhcpv4SrvTest, lease4RenewSimple) - modified to verify
        query4 set by lease4_renew callout

    src/lib/dhcpsrv/alloc_engine.cc
        - AllocEngine::createLease4()
        - AllocEngine::reuseExpiredLease4() - modified to add query4 to
        lease4_select callout arguments
        - AllocEngine::renewLease4() - modified to add query4 to lease4_renew
        callout arguments

    src/lib/dhcpsrv/tests/alloc_engine_hooks_unittest.cc
        HookAllocEngine4Test() - modified to track query4 argument
        TEST_F(HookAllocEngine4Test, lease4_select) - modified to verify query4
        callout argument

Added "query6" to pkt6_send and lease6_select
    src/bin/dhcp6/dhcp6_hooks.dox
        Added query6 argument to pkt6_send and lease6_select documentation

    src/bin/dhcp6/dhcp6_srv.cc
        Dhcpv6Srv::processPacket(Pkt6Ptr& query, Pkt6Ptr& rsp) - modified to
        add query6 to the pkt_send callout arguments

    src/bin/dhcp6/tests/hooks_unittest.cc
        Revamped to track both query4 and response4 arguments
        TEST_F(HooksDhcpv6SrvTest, simplePkt6Send) - modified to
        verify query6 set by pkt6_send callout

    src/lib/dhcpsrv/alloc_engine.cc
        - AllocEngine::createLease4()
        - AllocEngine::reuseExpiredLease6() - modified to add query6 to
        lease6_select callout arguments
        callout arguments

    src/lib/dhcpsrv/tests/alloc_engine_hooks_unittest.cc
        HookAllocEngine6Test() - modified to track query6 argument
        TEST_F(HookAllocEngine6Test, lease4_select) - modified to verify
        query6 argument

[master] Corrections in the config examples.

[master] ChangeLog updated after 4112 merge.

[master] Merge branch 'trac4112' (subnet4o6 selection based on v6 info)

[4112_rebase] Compilation fix.

[4485] Change type of variables to overcome compiler objections

One compiler was objecting to placing the address of a size_t variable
into a variable declared as "unsigned long*".  Changing the
declaration from "size_t" to "unsigned long" fixes the problem.

[4112_rebase] Minor change after review

 - check for address being :: updated

[master] Added ChangeLog entry 1112 for #4492.

[master] Fixed unit test compilation post merge

    HooksManager::loadLibraries() call now requires
    LibraryCollection as parameter

[master]  Server destructors now explicitly unload hooks libraries

    Merge branch 'trac4492'

[4492] Addressed review comments

src/bin/dhcp4/tests/hooks_unittest.cc
src/bin/dhcp6/tests/hooks_unittest.cc
    Fixed includes, copyright, and removed unnecessary calls to
    IfaceMgr::instance().deleteAllExternalSockets();

[4492] Addressed review comments

src/bin/dhcp4/tests/hooks_unittest.cc
    TEST_F(LoadUnloadDhcpv4SrvTest, unloadLibaries)  - new test that
    verifies Hooks libraries are unloaded by the Dhcpv4Srv destructor

src/bin/dhcp6/tests/hooks_unittest.cc
    TEST_F(LoadUnloadDhcpv6SrvTest, unloadLibaries)  - new test that
    verifies Hooks libraries are unloaded by the Dhcpv4Srv destructor

[4112] 2 new unit-tests added.

[master] Minor correction in doxygen comments.

 Thanks to wpk for reporting this issue.

[4112] selectSubnet4o6 implemented + one unit-test

[master] Added ChangeLog entry for #4493.

[master] Merge branch 'trac4493'

[4493] Optimize lookup of timed out packets.

[4493] Updated statistics calculation.

Presented rate is now the number of ACK/Reply messages,
rather than DO/SA.

[master] ChangeLog updated after #4297 merge.

[master] Merge branch 'trac4297' (hook libs can now take parameters)

[4493] Extended perfdhcp test for collected packets.

The extended test verifies that the stats manager is
still usable after some packets have been collected.

[4493] Improve packet collecting mechanisms in perfdhcp.

When the perfdhcp performs unordered packet lookup within
a bucket of sent packets, it will remove all timed out
transactions.

[master] Corrected invalid parameter passed to the test function.

This is to eliminate some of the recently found covertity isssues.
Reviewed on jabber.

[4492] Explicitly unload hook libraries prior during shutdown

A call to HooksManager::getHooksManager().unloadLibraries() was added to
the destructors Dhcpv4Srv::~Dhcpv4Srv() and Dhcpv6Srv::~Dhcpv6Srv(). This
ensures hook libraries are unloaded prior to static objects.

[4297] spelling

[4303] Added Changelog entry for #4303.

[4297] Corrected headers installation dir.

[4297] Addressed review comments:
 - returnining different values in callout_params_library.cc
 - returned values are now described for getParameter
 - corrected index syntax comment in getParameter
 - added missing include
 - hooks_user.dox updated
 - hooks unit test moved, so it is now next to other hooks tests

[master] Merge branch 'trac4303'

[4303] Removed unnecessary variable initialization.

[4303] Address second round of review comments.

- No additional check for HR mode in findReservations
- Additional commentary in the loop collecting host ids.

[master] Added ChangeLog entry 1108 for #4239

[master] Unit tests now use production database scripts

    Merges in trac4239.

[master] Updated ChangeLog

[master] regen

[4303] Added commentary in dhcp4_srv.h

[master] Finish merge of trac4268a (DHCPv4 fields)

[4303] Addressed remaining review comments.

[4303] Renamed CfgHostReservations to CfgHostOperations.

This change was suggested during a ticket review.

[trac4268] Tidy up some documentation strings

[4268a] Fixes and cleanups

[4268a] Updated tests for 4 byte padding

[4268a] untabify

[4268a] Padded hlen/htype pkt4

[4239] Addressed review comments

src/lib/dhcpsrv/pgsql_lease_mgr.cc
    Amended commentary for getColumnValue() variants
    Added decription of Uiaid union
    Changed std::system to ::system

src/lib/dhcpsrv/testutils/mysql_schema.cc
    Changed std::system to ::system

src/lib/dhcpsrv/testutils/mysql_schema.h
    Fixed typo

src/lib/dhcpsrv/testutils/pgsql_schema.cc
    Removed extraneous include
    Changed std::system to ::system

[4268a] Added extracted field lengthes

[master] Update changelog with commit number for 4269

Merge branch 'trac4269' Merge 4269 after updates to fix conflicts

[trac4269] Updates per review comments, basically change some variable names

[4303] Values in 'host-reservation-identifiers' used by the servers.

Both DHCPv4 and DHCPv6.

[4303] Implemented parser for 'host-reservation-identifiers' list.

[4303] Added class holding configuration of host identifiers.

The identifiers are held on an ordered list and the server will
use them to search for host reservations for each client using
this order.

[4303] Servers now search for host reservations using any identifier.

[4239] Moved schema scripts to src/share/database/scripts

    Moved schema scripts into a new common location:

        src/share/database/scripts

    to make them accessible to both kea-admin and dhcpsrv testing

[master] Added ChangeLog entry for pull request #19.

[master] Merge branch 'github19'

[github19] Updated AUTHORS file.

Added Angelo as contributor.

[4239] Added drop scripts for both MySQL and Postgresql

Rather than use hard-coded lists or query logic for dropping the
database all MySQL and Posgresql tests use new drop scripts
added to src/bind/admin/scripts.

src/bin/admin/scripts/mysql/dhcpdb_drop.mysql
    - New SQL script to drop a MySQL database

src/bin/admin/scripts/pgsql/dhcpdb_drop.pgsql
    - New SQL script to drop a Postgresql database

src/bin/admin/tests/mysql_tests.sh.in
    - mysql_wipe() - modified to use new drop script

src/bin/admin/tests/pgsql_tests.sh.in
    - pgsql_wipe() modified to use new drop script

src/lib/dhcpsrv/tests/mysql_lease_mgr_unittest.cc
    - TEST(MySqlOpenTest, OpenDatabase) - added show_err=true
    to destroyMySQLSchema() calls

src/lib/dhcpsrv/tests/pgsql_lease_mgr_unittest.cc
    - TEST(PgSqlOpenTest, OpenDatabase) - added show_err=true
    to destroyMySQLSchema() calls

src/lib/dhcpsrv/testutils/mysql_schema.cc
    - destroyMySQLSchema(bool show_err) - modified to use the new
    drop script and accept show_err parameter

src/lib/dhcpsrv/testutils/pgsql_schema.cc
    - destroyPgSQLSchema(bool show_err) - modified to use the new
    drop script and accept show_err parameter

[4239] MySQL and Postgresql unit tests use production schema create scripts

src/lib/dhcpsrv/testutils/schema.h
src/lib/dhcpsrv/testutils/schema.cc
    New files that contain constants and functions common to RDBMS
    testing

src/lib/dhcpsrv/testutils/pgsql_schema.h
src/lib/dhcpsrv/testutils/pgsql_schema.cc
    New files that contain constants and functions needed for
    Posgresql testing

src/lib/dhcpsrv/testutils/mysql_schema.cc
src/lib/dhcpsrv/testutils/mysql_schema.h
    Removed common constants and functions
    Added MYSQL_VALID_TYPE
    runMySQLScript() - new function to submit a script to MySQL

src/lib/dhcpsrv/pgsql_lease_mgr.cc
    PgSqlLease6Exchange - replaced uint32_t iaid member with Uiaid union
    instance.  This permits the value to be safely stored in the database
    as an INT (signed 4-byte value)
    getColumnValue()- added int32_t variant of this method

src/lib/dhcpsrv/tests/Makefile.am
    Removed schema_pgsql_copy.h

src/lib/dhcpsrv/tests/mysql_host_data_source_unittest.cc
src/lib/dhcpsrv/tests/mysql_lease_mgr_unittest.cc
    Altered tests to use MYSQL_VALID_TYPE

src/lib/dhcpsrv/tests/pgsql_lease_mgr_unittest.cc
    Moved constants and functions testutils/pgsl_schema.h & cc

[github19] Integrated pull request 19 with unit test fix.

[4239] Fix constraint-based unit test failure

src/lib/dhcpsrv/tests/generic_lease_mgr_unittest.cc
   - GenericLeaseMgrTest::testLease6HWTypeAndSource() - now uses valid values
     for hardware address source

[4239] MySQL unit tests in dhcpsrv now use production db create script

src/lib/dhcpsrv/tests/mysql_lease_mgr_unittest.cc
    TEST(MySqlOpenTest, OpenDatabase) - pass in show_err=true for call
    to createMySQLSchema()

src/lib/dhcpsrv/testutils/Makefile.am
    - Added definition of TEST_ADMIN_SCRIPTS_DIR
    - removed schema_mysql_copy.h

src/lib/dhcpsrv/testutils/mysql_schema.cc
    - Removed include of schema_mysql_copy.h
    - destroyMySQLSchema() - now contains destroy_statment[] formerly
    defined in schema_mysql_copy.h
    - createMySQLSchema() - Creates the database through command line
    invocation of MySQL cli, mysql, passing it the production database
    creation script

[trac4269] Update to put 4 byte strings on the value stack for integers

[master] Update change log to add commit number for trac4265

[master] Added ChangeLog entry 1104 for #4259.

[master] DDNS parameter, replace-client-name, now supports multiple modes

    Merges in trac4259.

[4259] Address review comments, part II

    Fixed minor typos.

[master] Added ChangeLog entry for #4301.