Storing payloads with the related serialization info. Improved error … #9013
Conversation
…handling in serialization.
|
+1 |
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_5_X IBs unless changes or unless it breaks tests. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @nclopezo, @smuzaffar |
|
-1 >> Compiling /build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_7_5_X_2015-05-07-2300/src/Alignment/LaserAlignment/src/LASPeakFinder.cc
In file included from /afs/cern.ch/cms/sw/ReleaseCandidates/vol1/slc6_amd64_gcc491/external/clhep/2.1.4.1-cms/include/CLHEP/Vector/EulerAngles.h:19:0,
from /build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_7_5_X_2015-05-07-2300/src/CondFormats/Alignment/interface/AlignTransform.h:5,
from /build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_7_5_X_2015-05-07-2300/src/CondFormats/Alignment/interface/Alignments.h:6,
from /build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_7_5_X_2015-05-07-2300/src/Alignment/CocoaFit/src/CocoaDBMgr.cc:1:
/afs/cern.ch/cms/sw/ReleaseCandidates/vol1/slc6_amd64_gcc491/external/clhep/2.1.4.1-cms/include/CLHEP/Vector/defs.h:41:17: error: expected unqualified-id before string constant
#define VERSION "2.1.4.1"
^
/build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_7_5_X_2015-05-07-2300/src/CondCore/CondDB/interface/Serialization.h:95:34: note: in expansion of macro 'VERSION'
static constexpr char const\* VERSION = BOOST_LIB_VERSION;
^
you can see the results of the tests here: |
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_5_X IBs unless changes (but tests are reportedly failing). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @nclopezo, @smuzaffar |
|
+1 |
|
@cmsbuild |
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_5_X IBs unless changes or unless it breaks tests. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @nclopezo, @smuzaffar |
cmsbuild
removed
comparison-pending
db-approved
fully-signed
orp-pending
tests-pending
labels
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_5_X IBs unless changes (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @nclopezo, @smuzaffar |
|
+1 |
Storing payloads with the related serialization info. Improved error …
…handling in serialization.
This PR is introducing the serialization info in the storage of payloads. The errors in the serialization/de-serialization are interpreted and re-thrown.