datopian / metastore-lib

🗄️ Library for storing dataset metadata, with versioning support and pluggable backends including GitHub.
https://tech.datopian.com/versioning/
MIT License
10 stars 1 forks source link

[performance] no need to fetch base revision during `update` if not doing a partial update #21

Closed shevron closed 4 years ago

shevron commented 4 years ago

As we learned from #17, even when update is full and not partial, we are still fetching the "base" revision for an update, but this information is really not used in the update code unless the update is partial.

We should just not fetch the base revision if the update is not partial.

shevron commented 4 years ago

Resolved in #25