search

ARPA-SIMC / dballe

Fast on-disk database for meteorological observed and forecast data.
Other
19 stars 6 forks source link

Remove dballe.DB.{query,insert,remove}* #196

Closed edigiacomo closed 1 year ago

edigiacomo commented 5 years ago

As proposed in #146, we have already added the deprecation warning to these methods. Now we have to choose the version from which remove them.

pat1 commented 4 years ago

rimuoviamolo anche da ogni esempio !

spanezz commented 4 years ago

L'unica menzione che sono riuscito a trovare era in volnd, e l'ho sostituita. Se ne sapete altre mandatemi i link

pat1 commented 4 years ago

Non so se interpreto bene la richiesta ma io la intendo così:

Many methods are the same in dballe.DB and dballe.Transaction. The versions in dballe.DB are implemented by automatically creating a temporary transaction and running the equivalent dballe.Transaction method inside it.

questo non dovrebbe essere più vero e dballe.DB non dovrebbe creare una transazione temporanea esplicitato che TUTTI quei metodi vanno chiamati da una transazione e non direttamente ... perchè al momento sono richiamabili direttamente, spesso non funzionano e creano grande confusione

edigiacomo commented 1 year ago

Questa issue è stata chiusa per inattività. Nel caso in cui sia un argomento ancora rilevante, si prega di riaprirla con una motivazione che tenga conto delle modifiche applicate nel corso degli anni al progetto.