Skip to content

Fix CCDB access in TPC PID response handler tool#1020

Merged
iarsene merged 2 commits into
AliceO2Group:masterfrom
jezwilkinson:TPChandler
Aug 1, 2022
Merged

Fix CCDB access in TPC PID response handler tool#1020
iarsene merged 2 commits into
AliceO2Group:masterfrom
jezwilkinson:TPChandler

Conversation

@jezwilkinson

Copy link
Copy Markdown
Contributor

TPC PID response handler's CCDB access was broken by a previous commit that modified the timestamp initialisation in handleParamBase.h (#934). This PR fixes the incompatibility and removes redundancies with the options that were added to the base header.

Only impacts the parameter object handler tool; the analysis tasks are unaffected.

@jezwilkinson jezwilkinson requested review from a team, alibuild, iarsene, jgrosseo and ktf as code owners July 19, 2022 13:58
@alibuild

alibuild commented Jul 20, 2022

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for 4da436a at 2022-07-27 04:38:

## sw/BUILD/O2Physics-latest/log
c++: fatal error: Killed signal terminated program cc1plus
ninja: build stopped: subcommand failed.

Full log here.

@jezwilkinson

Copy link
Copy Markdown
Contributor Author

Hi @jgrosseo @iarsene @ktf - could you please check if the pull is OK? We can work around the issue by using the locally built version of the handler to push to CCDB, but it would be preferable to have the central version working properly.
Thanks, --Jeremy

@iarsene iarsene merged commit f369b12 into AliceO2Group:master Aug 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants