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.

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

Escalate targeted graph refresh to full graph refresh #15035

Closed
durandom opened this issue May 9, 2017 · 5 comments
Closed

Escalate targeted graph refresh to full graph refresh #15035

durandom opened this issue May 9, 2017 · 5 comments
Assignees
Labels

Comments

@durandom
Copy link
Member

durandom commented May 9, 2017

a targeted refresh of any CI that has no targeted refresh implemented should be escalated to a full graph refresh.

see discussion in #15025 (comment)

I think this fallback should be removed completely and the escalation should happen before, maybe here ?

@miq-bot assign @durandom

cc @Ladas

@Ladas
Copy link
Contributor

Ladas commented May 9, 2017

@durandom right, so if you pass unknown AR object that has no Persister defined, it will go to 'next' and skip the target I think.

@miq-bot
Copy link
Member

miq-bot commented Nov 11, 2017

This issue has been automatically marked as stale because it has not been updated for at least 6 months.

If you can still reproduce this issue on the current release or on master, please reply with all of the information you have about it in order to keep the issue open.

Thank you for all your contributions!

@miq-bot miq-bot added the stale label Nov 11, 2017
@JPrause
Copy link
Member

JPrause commented Jan 23, 2019

@Ladas is this still a valid issue. If not can you close.
If there's no update by next week, I'll be closing this issue.

@Ladas
Copy link
Contributor

Ladas commented Jan 23, 2019

I think this is solved, we can close

@JPrause
Copy link
Member

JPrause commented Jan 23, 2019

Thanks @Ladas
@miq-bot close_issue

@miq-bot miq-bot closed this as completed Jan 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants