cmu-delphi/delphi-epidata

add a runner and test(s) for _reset_load_table_ai_counter

Open

#991 opened on Oct 4, 2022

View on GitHub
 (0 comments) (0 reactions) (0 assignees)Python (67 forks)auto 404
acquisitioncode healthdevopsgood first issuehelp wantedtests

Repository metrics

Stars
 (104 stars)
PR merge metrics
 (PR metrics pending)

Description

as seen in: https://github.com/cmu-delphi/delphi-epidata/blob/dev/src/acquisition/covidcast/database.py#L126

  • also make change to ensure epimetric_load is empty first instead of just blindly clearing it

motivations for actual implementation reset process:

extra credit:

  • research (and implement?!?) if can we deny ability to TRUNCATE epimetric_load through permissions? (--> ie, remove drop permissions for epimetric_* family of tables to prevent resetting of autoincrement ids again)
  • verify that we do not replicate epimetric_load table, though this is not a deal-breaker

Contributor guide

Research direction
Implement a test for the ` reset load table ai counter` function and modify the code to ensure `epimetric load` is empty before resetting. Also consider the extra credit items: research permissions to deny TRUNCATE, and verify replication settings.
Tech stack
pythonmysql
Domain
backenddatabase
Issue type
Test
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
GitPythonMySQL
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70