petl-developers / petl / 34303235165

92%

Build Type

push

github

Committed by web-flow

Commit Message

Preserve record metadata across pickle round trips (#712)

Fixes #299.

`Record` needs field names when reconstructed, but tuple's default
pickle path does not supply them. Older pickle protocols also probe
attributes before field metadata is restored, causing recursive
`__getattr__` calls. Define the record reconstruction arguments and
preserve instance state; access field metadata directly when resolving
attributes so older protocol-0 data can be restored.

The normal test suite now covers all available pickle protocols,
short/full/extended rows, named and positional access, missing values,
extra metadata, older serialized data, and a real `recordlookup()` round
trip through a closed and reopened `shelve` database. Document the
serialization contract in `records()`.

Validation: 19 new round-trip/shelve cases failed before the fix, and
the additional legacy fixture exposed the attribute recursion. The final
utility/transform suite passes: 374 tests, 1 skipped. Sphinx HTML builds
with warnings treated as errors. External database/remote filesystem
tests were not run.

Coverage Stats

41 of 41 new or added lines in 2 files covered. (100.0%)

8 existing lines in 2 files now uncovered.

14239 of 15466 relevant lines covered (92.07%)

0.92 hits per line