Shawn Walker wrote:
Greetings,

The following webrev contains fixes for the following issues:
2557 catalog cache pickle file dependent on fmri and version object definitions
  6221 AttributeError while reading catalog.pkl

webrev:
http://cr.opensolaris.org/~swalker/pkg-2557/

Change Summary:
* Completely removed all pickle usage in favor of custom format flat file for catalog cache (see comments for format details)

* Added catalog.fast_cache_fmri() method to skip the extra overhead of consistency checks and sorting for cache loads

* Added a set of Catalog cache exceptions to allow more specific analysis of cache failure

* Bumped catalog_cache_version to 4

* Made catalog cache parsing robust against corruption, truncation, or invalid values

* Added code to remove old catalog cache files when possible (pkg_names.pkl, catalog.pkl)

* Added code to ensure that if catalog cache can't be read or is corrupt that an in-memory representation will be built anyway from the existing catalog data and an attempt to save it will be made (although failure will not occur if it cannot be).

* Added unit tests to verify that corrupt or missing catalog cache will not impact pkg operations such as pkg list.

Alright, I've incorporated all the review feedback, and I've altered the flat-file format a little further for better performance and a reduction in size:
http://cr.opensolaris.org/~swalker/pkg-2557-2/

new catalog_cache sample:
http://cr.opensolaris.org/~swalker/pkg-2557-2/catalog_cache

I need one more reviewer.

Cheers,
--
Shawn Walker
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to