cmu-delphi/delphi-epidata
add a runner and test(s) for _reset_load_table_ai_counter
オープン
#991 opened on 2022/10/04
acquisitioncode healthdevopsgood first issuehelp wantedtests
Repository metrics
- Stars
- (104 個のスター)
- PR merge metrics
- (平均マージ 63d 20h) (30d で 11 merged PRs)
説明
as seen in: https://github.com/cmu-delphi/delphi-epidata/blob/dev/src/acquisition/covidcast/database.py#L126
- also make change to ensure
epimetric_loadis empty first instead of just blindly clearing it
motivations for actual implementation reset process:
- "ALTER TABLE ... AUTO_INCREMENT = N can only change the auto-increment counter value to a value larger than the current maximum."
- "To reset the current auto-increment value ... You cannot reset the counter to a value less than or equal to the value that is currently in use"
extra credit:
- research (and implement?!?) if can we deny ability to TRUNCATE
epimetric_loadthrough permissions? (--> ie, remove drop permissions forepimetric_*family of tables to prevent resetting of autoincrement ids again) - verify that we do not replicate
epimetric_loadtable, though this is not a deal-breaker