Removed non-sqlite database support for results/tracking Db

lsst-sims / legacy_sims_maf

LSST Simulations package for the metrics analysis framework (MAF)

13 stars 19 forks source link

Removed non-sqlite database support for results/tracking Db #187

Closed rhiannonlynne closed 4 years ago

rhiannonlynne commented 4 years ago

We have never used alternative databases for the results and tracking databases than local sqlite files, so I think it's just fine to remove the dbAuth support for other formats.

rhiannonlynne commented 4 years ago

Thanks @jdswinbank for the review. It's nice to send one of these to someone other than @yoachim every once in a while and this seemed like a good one, for letting you know how we resolved the pex_persistence issue. We have never actually used the non-sqlite capabilities for the two classes I changed, so I'm sure it's ok .. and I'm 100% sure that @yoachim will agree on dumping them.