-
Notifications
You must be signed in to change notification settings - Fork 483
DPL CCDB: avoid duplicated code #14560
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
REQUEST FOR PRODUCTION RELEASES: This will add The following labels are available |
|
Error while checking build/O2/fullCI_slc9 for de9dbc8 at 2025-08-07 18:25: Full log here. |
|
@singiamtel any idea why it's now failing the clang-tidy part? IIRC you changed something related? |
| O2_SIGNPOST_EVENT_EMIT(ccdb, sid, "populateCacheWith", "Caching %{public}s for %{public}s (DPL id %" PRIu64 ")", path.data(), headers["ETag"].data(), cacheId.value); | ||
| continue; | ||
| } | ||
| if (v.size()) { // but should be overridden by fresh object |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@shahor02 elsewhere we have a similar pattern, but using an else if rather than a separate else statement. Is that actually expected to be different here?
The change I made was trying to fix that exact error, I guess it didn't work... It's O2-6169 |
|
Error while checking build/O2/fullCI_slc9 for 9fa4c3d at 2025-08-16 02:42: Full log here. |
|
Will do it differently... |
No description provided.