Removed the FileName attribute in the dlf message 'Error retrieving the file stat from the nameserver' as it is always NULL

  TapeAccessSpecification.accessMode DESC,
  VolumePriority DESC,
  TapeRequest.modificationTime ASC;

Fixed filesDeletedProc. Previous implementation was not deleting the castorfile, nor reporting about files to be dropped from the nameserver

Fixed bind bug in deleteVolPriority.  Thesize of the clientHostVar string was not given explicity and the default was too small and would cause a ORA-06502: PL/SQL: numeric or value error ORA-06512

Added a log message which explicitly states when a volume priority has not been deleted at the begin of the unmount process

Improved the signature of method castor::vdqm::IVdqmSvc::deleteVolume, by making the constant pointers explicit.

the corresponding tape is started.

Added castor::db::ora::OraVdqmSvc::deleteColPriority.  This method will eventually be called to end the lifecycle of a volume priority.  When exactly to make that call is still being worked out.

Added triggers to keep the creationTime and modificationTime columns of the VolumePriority table upto date.  Added NOT NULL constraints for VolumePriority table

allow compilation of the new stagerJob. Note, for the time being it won't be compiled as STAGERJOBDIR is not defined in Project.tmpl

to be tested.

or "close but no cigar".  Yes I broke the build... Sorry!

One can use a constant as the size of a variable outside of a function with
C++, as the following code which compiles successfully with -pedantic -ansi
demonstrates:

static const unsigned int ObjectsIdsNb = 152;

const char* a[ObjectsIdsNb];

int main(int argc, char **argv) {
  using namespace std;

  cout << "Hello World!" << endl;
  cout << "sizeof(a)/sizeof(char*)=" << sizeof(a)/sizeof(char*) << endl;
}

Unfortunately the C compiler, as opposed to the C++ compiler, does not allow a
variable-size type declared outside of any function.

In order to keep a solution where the numbers of object, service and
repriesentation IDs are only written once by the programmer, I have added a
macro for each of these values whilst preserving the C++ constants.  For
example, in the case of ObjectsIdsNb the code now reads:

  /**
   * Nb of objectIds currently existing
   *
   * Note that both a macro and a constant are required because the C compiler,
   * as opposed to the C++ compiler, does not allow a variable-size type
   * declared outside of any function.
   */
#define OBJECT_IDS_NB 152
  static const unsigned int ObjectsIdsNb = OBJECT_IDS_NB;

Added the new VDQM2 object type id: OBJ_VolumePriority, plus used the constants ObjectsIdsNb, ServicesIdsNb, and RepresentationsIdsNb to set the size of their corresponding string arrays, i.e. ObjectsIdStrings, ServicesIdStrings and RepresentationsIdStrings

  1. castor::db::ora::OraCommonSvc is in castor/db/newora
  2. castor::db::ora::OraCommonSvc contains selectTape
  3. selectTape doesn't just select tape rows, it creates them if they cannot
     be selected
  4. castor::db::ora::OraCommonSvc::selectTape copied into
     castor::db::ora::OraVdqmSvc as VDQM2 code should be independent of the
     stager

I have changed the name of castor::db::ora::OraVdqmSvc::selectTape to
castor::db::ora::OraVdqmSvc::selectOrCreateTape. FYI castor::db::ora::OraVdqmSvc
is in castor/db/ora.

Removed setVolumePriority() from TapeRequestHandler.  I have decided to keep the handling of new VDQM_MAGIC2 messages within the new class VdqmMagic2RequestHandler.

local variables used to store the same values and the formal function
arguments used to pass them.