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

Redis counter does not increases with duplicate data from regulations #165

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AlexF301
Copy link
Contributor

@AlexF301 AlexF301 commented Apr 21, 2023

attempt to fix redis counter going up when duplicate data comes through from regulations. Made a callback function in Client which gets passed onto DiskSaver. when disksaver finds the path for the job doesn't exist, calls the callback function from Client, increasing the redis counter.

attempt to fix redis counter going up when duplicate data comes through from regulations.
Made a callback function in Client which gets passed onto DiskSaver. when disksaver
finds the path for the job doesn't exist, calls the callback, increasing the redis counter.

Have to write code/test for the non-happy path
@AlexF301 AlexF301 changed the title Redis counter only increases with new data Redis counter does not increases with duplicate data from regulations Apr 21, 2023
@AlexF301
Copy link
Contributor Author

Not sure if this was the best way to do this. feedback is appreciated

@jpappel
Copy link
Collaborator

jpappel commented Oct 29, 2024

@bjcoleman This seems relevant to the dashboard bug

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.

2 participants