1082 Calculate distribution size #1489
Conversation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
kastger
changed the title
| distributions_with_same_url = self.dataset.datasetdistribution_set.filter( | ||
| download_url=url | ||
| distributions_with_same_url = ( | ||
| self.dataset.datasetdistribution_set.filter(download_url=url) | ||
| ) | ||
| if distributions_with_same_url.exists(): |
There was a problem hiding this comment.
bet pagal tą nuorodą gali būti ir atnaujintas failas, tai tada jį reikėtų parsiųsti, kaip suprantu
There was a problem hiding this comment.
Sprendimas neatitinka užduoties:
Katalogas periodiškai, tikrindamas ar failai pasiekiami, turėti atsisiųsti failą ir apskaičiuoti jo dydį.
-
kur yra periodiškumas? Gal jis jau buvo anksčiau, bet nesimato pakeitimuose? Jei ne, tai ar jis įgyvendinamas, ar ne?
-
dabar tiesiog tikrinama, ar tas failas jau yra, ir jei jis jau yra, nesiunčiama. Ir tikrinama tik įvedimo metu.
Bet pagal užduotį, reikia periodiškai tikrinti, ir net jei failas yra tuo pačiu pavadinimu, reikia papildomai patikrinti Last-Modified ir jei jis atsinaujinęs, atsisiųsti ir perskaičiuoti dydį iš naujo.
Tai periodinis tikrinimas turėtų būti daromas arba su cronjob, arba su celery, jei jis projekte yra.
Failas turėtu būti siunčiamas tik tuo atveju, jei jo turinys pasikeitė, ar pasikeitė turinys gali būti nustatoma patikrinant atitinkamas HTTP antraštes, kurios nurodo ar resursas pasikeitė.
Nėra tikrinama, ar failas pasikeitė, ir nėra saugoma paskutinio pasikeitimo data.
Prie užduoties pridėjau priėmimo kriterijus.
Example: