Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Deleting a bundle should not touch the PushedAsset Cache #18025

Closed
wezell opened this issue Feb 22, 2020 · 2 comments
Closed

Deleting a bundle should not touch the PushedAsset Cache #18025

wezell opened this issue Feb 22, 2020 · 2 comments
Assignees
@jgambarios
Labels
Changelog: Documented Merged QA : Passed Internal Release : 5.2.8 Type : Defect

Comments

@wezell
Copy link
Contributor

wezell commented Feb 22, 2020

I see this in the logs on one of our servers, over and over.

https://gist.github.com/wezell/4357f88c3270b9f29760993ba9a46504

I looked in our code and 2 things

  1. deleting a bundle should not clear the PushedAssetCache
  2. The clearCache method which is being called is synchronized? Why?
@wezell wezell added this to the Bug Sprint milestone Feb 22, 2020
@fabrizzio-dotCMS fabrizzio-dotCMS self-assigned this Mar 3, 2020
fabrizzio-dotCMS added a commit that referenced this issue Mar 9, 2020
@fabrizzio-dotCMS
#18025 deleting a bundle should not delete any pushedAssets
31f56aa
@fabrizzio-dotCMS fabrizzio-dotCMS mentioned this issue Mar 9, 2020
Merged
@jgambarios
Copy link
Contributor

PR: #18118

jgambarios pushed a commit that referenced this issue Mar 9, 2020
@fabrizzio-dotCMS
#18025 deleting a bundle should not delete any pushedAssets (#18118)
a933c1b
@jgambarios
Copy link
Contributor

Fixed, publishing_pushed_assets records or the related cache are not deleted anymore when a bundle is deleted or when a Status is deleted from the Publishing Queue -> Status/History tab.

jgambarios pushed a commit that referenced this issue Mar 10, 2020
@fabrizzio-dotCMS @jgambarios
#18025 deleting a bundle should not delete any pushedAssets (#18118)
6f6b993 
(cherry picked from commit a933c1b)
@wezell wezell closed this as completed Apr 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jgambarios jgambarios

Labels
Changelog: Documented Merged QA : Passed Internal Release : 5.2.8 Type : Defect
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@jgambarios @wezell @john-thomas-dotcms @fabrizzio-dotCMS