cmu-delphi/delphi-epidata

clean up insert_or_update_batch() method

Aperta

#989 aperta il 4 ott 2022

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Python (67 fork)auto 404
acquisitioncode healthgood first issuehelp wantedrefactor

Metriche repository

Star
 (104 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

  • remove insert_or_update_bulk()
  • roll run_dbjobs() logic into the insert_batch method
    • can get rid of "suppress_jobs" argument too, unless someone can come up with a good reason for keeping it

https://github.com/cmu-delphi/delphi-epidata/blob/dev/src/acquisition/covidcast/database.py#L159

Guida contributor

Direzione di ricerca
Ispeziona il file sorgente collegato (src/acquisition/covidcast/database.py) per comprendere l'attuale metodo insert or update batch() e le funzioni correlate insert or update bulk() e run dbjobs(). Quindi rimuovi insert or update bulk() e integra la logica di run dbjobs() in insert batch, gestendo opportunamente l'argomento 'suppress jobs'.
Tech stack
python
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PythonGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75