rpc_version -> protocol_version

e.g.
sacctmgr modify federation <fed> set clusters=

ex.
sacctmgr add cluster <cluster(s)> federation=<fed_name>
sacctmgr modify cluster <cluster(s)> federation=<fed_name>
sacctmgr add federation clusters=<federation_names>

Goes on the cluster instead.

Problem introduced in commit 0a4d5770

NOTE: Runaway jobs are jobs that don't exist in the controller but are still considered running in the datbase
Should be
NOTE: Runaway jobs are jobs that don't exist in the controller but are still considered running in the database

# Conflicts:
#	src/plugins/jobcomp/elasticsearch/jobcomp_elasticsearch.c

(difference between start of job and when it was eligible).

match that of other plugins.  Send in a pointer to a structure that gets
allocated and then filled in instead of sending in an already
allocation structure.

Had chance to work on DW testbed and fix problems discovered.

Bug 2939.

list.c:987: warning: ‘list_mutex_is_locked’ defined but not used
cbuf.c:1684: warning: ‘cbuf_mutex_is_locked’ defined but not used
Use "#ifndef NDEBUG" to fix and add "_" suffix on static function names.

Swap max_time and total_time fields, which seems better to me

This reverts commit fb8e3558
and moves the place where the SuspendExcNodes and SuspendExcParts
configuration parameters are processed (needs to happen AFTER the
partition and node tables in the slurmctld daemon are built.
bug 2934