Given CORAL transaction context seems to be global, one thread can
commit a transaction, which is active at that moment in another thread.
Worked around this issue by starting readonly transaction when a
session is created, and committing that transaction only when the
corresponding session is closed.

Addresses the issue discussed in ATLASRECTS-5737.

Also removed several redundant include statements.

Also delete unused (broken) `RegistrationStream.py`.

IOVDbSvs patched such that if IOV for a given folder is overriden,
then it sets infinite validity for the condition object fetched
from the database. As a result, the folder cache never gets updated, and the
object fetched from the database remains valid for the duration of the job.

Addresses ATEAM-649

Rachid Mazini authored 4 years ago and Atlas Nightlybuild committed 4 years ago

Modify the pers->trans conversion for cool based metadata

See merge request atlas/athena!37346

(cherry picked from commit 82513fc8229253bed05ddf416066ef889948f923)

9f1a55d0 Modify the pers->trans conversion

Oana Vickey Boeriu authored 4 years ago and Atlas Nightlybuild committed 4 years ago

Set container splitlevel 1 for DAOD PHYSLITE

See merge request atlas/athena!36372

(cherry picked from commit c790ca4bdca5b36fc4e4ae5013225f42f6d61938)

a33831bb set container splitlevel 1 for phys/physlite
297597f1 only physlite
438e5cbd set to splitlevel 99

This property accepts comma-separated list of Event and Event Ranges
that the EventSelector will skip when reading from the input collection.
Implemented as a new property because the SkipEventSequences uses a list
of integers and changing that would break existing joboptions.

Due to a CORAL bug, we had declared a number of functions here
as not thread-safe.  The underlying CORAL bug has now been fixed,
so we can now remove these annotations.

 - Pass class instances by const reference, not by value.
 - Don't use string::find to test the start of a string.
 

 - Pass class instances by const reference, not by value.

 - Pass class instances by const reference, not by value.

 - Prefer pre-increment (or range-for) to postincrement.
 - Pass class instances by const reference, not by value.

The thraed-safety problems of AttributeList have now been fixed in CORAL.
Remove not-thread-safe annotations from AthenaAttributeList.

 - Prefer pre-increment (or range-for) to postincrement.
 - Prefer using an initialization list to initializing membesr
   in a ctor body.

 - Prefer pre-increment (or range-for) to postincrement.
 - Pass class instances by const reference, not by value.
 - Fix out-of-bounds reference.
 

Mark calls to CollectionFactory / CollectionService as not thread-safe.

ICollectionMetadata::const_iterator now takes a unique_ptr.

ICollectionMetadata::const_iterator now takes a unique_ptr.

ICollectionMetadata::const_iterator now takes a unique_ptr.

Enable thread-safety checking.
Fix resulting warnings.
CollectionFactory / CollectionService marked as not thread-safe
due to const-correctness violations.

Enable thread-safety checking.
Fix resulting warnings.

    
Remove reference to obsolete header StoreGate.h.

    
Remove reference to obsolete header StoreGate.h.