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
feat: Add mysql staleness removal task #474
feat: Add mysql staleness removal task #474
Conversation
the delta extractor CI error is not related. I will take a look. |
@sahithi03 could you rebase the pr given #476 is now merged? this should fix the CI error. |
Signed-off-by: Sahithi Velma <sahithiv@wepay.com>
878b035
to
25cfd56
Compare
cc @crazy-2020 |
It looks good to me👍 |
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.
one nit, but lgtm
DRY_RUN: False, | ||
ENGINE_ECHO: False}) | ||
|
||
def __init__(self) -> None: |
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.
do we need this?
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.
No not needed, will update 👍
Signed-off-by: Sahithi Velma <sahithiv@wepay.com>
Signed-off-by: Sahithi Velma sahithiv@wepay.com
Summary of Changes
Adds a stale data removal task for mysql metadata store
Tests
Added unit tests for the mysql_staleness_removal_task
Documentation
N/A
CheckList
Make sure you have checked all steps below to ensure a timely review.
make test