sapling/blobstore
Zeyi Fan 82ab81238d re-exporting types from other crate is not a good idea
Summary:
I had enough seeing compiler giving me error messages with `blobrepo::Error` whereas it should be `failure::Error`.

`pub type` is exporting a type alias whereas `pub use` is re-exporting. The later apparently will fool the compiler to think that they own the thing they are re-exporting which in this case, failure::Error. If it is for the purpose of declaring the error type the crate is using, type alias is obviously more suitable as it won't take the ownership of the type.

Reviewed By: lukaspiatkowski

Differential Revision: D8926275

fbshipit-source-id: 6b1346adf52bec6b666a3f8b23e7a4ea4e0ab070
2018-07-26 14:23:07 -07:00
..
delayblob/src blobstore: implement Debug for all blobstores 2018-07-20 11:37:36 -07:00
fileblob/src blobstore: restrict the types that can be stored 2018-04-18 19:53:32 -07:00
if Implement a "cache mutex" to avoid multiple writes through memcache 2018-06-19 13:07:13 -07:00
rocksblob/src blobstore: implement Debug for all blobstores 2018-07-20 11:37:36 -07:00
src re-exporting types from other crate is not a good idea 2018-07-26 14:23:07 -07:00
test blobstore: move memblob into main blobstore 2018-06-12 15:40:10 -07:00
Cargo.toml mononoke: backout cachelib integration for blob caching 2018-07-26 10:09:32 -07:00