- Queueing into the RetrieveQueueToReportToRepackForFailure
- Reporting of the failed RetrieveJobs (add statistics + deletion of the failed sub requests)
Unit test associated : expandRepackRequestFailedRetrieve

These requests are queued in the RetrieveQueueToReportToRepackForFailure
Adapted the Sorter to be able to queue a RetrieveRequest from Repack into a queue identified by the RepackRequest's address

Updated the unit tests to validate the queues and requests are as expected.

Changed the reporting structures in the repack request to better handle repoting
(we will report one archive job per rearchived copy number, which is potentially
several per fSeq.

Created the getNextBatch() function for retrieve successes and partially implemented
the report function.

Also created the call in the maintenance process.

Reports can now be either "to user" or "to repack".

Renamed function to getNextRetrieveRequestToReportToRepackForSuccessBatch to match current naming convention.

The retrieve requests now carry all the information necessary to create the archive jobs.
Retrive requests carry an isRepack boolean. The distinction between repack and non-repack behaviour
is pushed as low as possible in the api (in the object store) so that client code can be more
generic. For the same reason, the
The expand repack mode (re-creation of missing tape files) is not supported yet.
Changed the represenation of copy numbers to uint32 (as this is the smalled number supported but protocol buffers).
Added ArchiveRoute::StorageClassMap and ArchiveRoute::FullMap to store archive routing in memory.
Added repack buffer base URL support, and mechanism to compute temporary copy location in disk cache.
Fixed bug in ObjectOps<>::asyncInsert();
Inplemented the path generation for repack disk storage (<BaseURL>/<VID>/<FSeq (09%d)>).

Renamed some status (#447 issue)
Harmonized the name of the Archive and Retrieve queues in the cta.proto file and made the changes in the C++ code

Harmonized the copyNb type to uint32_t
Corrected a bug in RetrieveQueue::getContainerSummary() --> After insertion in the queue, the number of files and bytes added were equal to 0.

Queueing of Archive Jobs is done and unit tested
Queueing of Retrieve Requests is not completely done yet

Started to queue the ArchiveRequest for Repack

It was using vid as container identifier where it should have used
tapepool.

Changed 'catch (std::out_of_range)' to 'catch (std::out_of_range&)' in order to avoid the error: catching polymorphic type ‘class std::out_of_range’ by value

Removed the "User" message from the cta.proto (duplication of UserIdentity message)

Previously these were references, but sometimes we want only an archive
queue itor or only a retrieve queue itor. Now we can set the unused
itors to nullptr.

Changed the return type of scheduler.getNextSucceededRetrieveRequestForRepackBatch() method from cta::SchedulerDatabase::RetrieveJob to cta::RetrieveJob

The getNextSucceededRetrieveRequestForRepackBatch is implemented and integrated in the expandRepackRequest test

Started to implement the method to get the RetrieveRequest from the RetrieveQueueToReportToSuccessForRepack queue

Refactored OStoreDB::RetrieveMount::batchSucceedRetrieveForRepack() method

Constructor takes a QueueType parameter so we can iterate across
FailedJobs in the same way as we do with JobsToTransfer.

-> Moved code related to payload manipulation in the cta::objectstore::RetrieveRequest class

Changed the operator "<" of Scheduler::PotentialMount so that the sorting of mounts is in alphabetical order

Changed the status of succeeded retrieve requests as RJS_Succeeded and inserted them to the RetrieveQueueToReportToRepackForSuccess, unit tested but memory leak

This was based on the assumption that queue would be trimmed fast, leading to a tight loop. We now go one round on existing queues.