Skip to content

MuDst corruption fix#360

Merged
starsdong merged 36 commits intostar-bnl:mainfrom
jdbrice:fcs-mudst-fix
Jun 8, 2022
Merged

MuDst corruption fix#360
starsdong merged 36 commits intostar-bnl:mainfrom
jdbrice:fcs-mudst-fix

Conversation

@jdbrice
Copy link
Copy Markdown
Contributor

@jdbrice jdbrice commented Jun 7, 2022

Fcs MuDst containers for hits and points were incorrectly storing references to fcs clusters, causing a crash on the second event when reading back a MuDst made with the full FCS chain:

fcsDat fcsWFF fcsCluster fcsPoint

jdbrice and others added 30 commits June 11, 2021 10:16
update from upstream on Friday, June 11, 2021
Updated StarGeo.xml to be able to acces this in dev2021
@plexoos
Copy link
Copy Markdown
Member

plexoos commented Jun 7, 2022

One test failed

StChain:INFO  - MysqlDb::Connect: Load balancer took 0 ms, will use dbx.star.bnl.gov:3316
StChain:INFO  - Ssl_cipher = AES128-SHA
StChain:ERROR - MysqlDb::ExecQuery() line=662  Query Failed  Lost connection to MySQL server during query
Error: Process completed with exit code 1.

I see a few other jobs completed successfully and presumably did not have a problem with connection to the MySQL server. Could be a glitch, so I'll restart the CI jobs.

Copy link
Copy Markdown
Member

@starsdong starsdong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@starsdong starsdong merged commit 1730019 into star-bnl:main Jun 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants