Joerg Stelzer authored 4 years ago and Vakhtang Tsulaia committed 4 years ago

* fix integer div
* remove flake UT
* overwrite default ignore list to enable E70x

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

Alaettin Serhan Mete authored 4 years ago and Vakhtang Tsulaia committed 4 years ago

Add a basic helper function to build PoolAttribute strings, switch to using it, revive the split level of 99 to POOLContainerForm(DataHeaderForm)

Marcin Nowak authored 4 years ago and Edward Moyse committed 4 years ago

Trying to group access to TagInfo object exclusively to the
TagInfoMgr so it can be better managed during input file transitions
in MT

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.