NotificationsYou must be signed in to change notification settings
Fork515
Star3.8k

Adds support for zero-copy FromSql derive#1070

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Open

PipelineBaron wants to merge4 commits intorust-postgres:master

base:master

Choose a base branch

fromPipelineBaron:zero-copy-fromsql-derive

Open

Adds support for zero-copy FromSql derive#1070

PipelineBaron wants to merge4 commits intorust-postgres:masterfromPipelineBaron:zero-copy-fromsql-derive

Conversation

Copy link

PipelineBaron commentedOct 11, 2023

Similar in spirit and implementation to howserde supports borrowed deserialization lifetimes. Likeserde, it uses'de as the lifetime of the trait (FromSql<'de>) and it adds all borrowed liftimes as lifetime bounds on'de.

Also likeserde all top level references automatically have their lifetimes borrowed, but container types carrying references must explicity be borrowed with the#[postgres(borrow)] annotation.

Follows up on:#574

PipelineBaron added3 commits

October 11, 2023 11:46

Adds support for zero-copy FromSql derive

309207a

Similar in spirit and implementation to how `serde` supports borroweddeserialization lifetimes. Like `serde`, it uses `'de` as the lifetimeof the trait (`FromoSql<'de>`) and it adds all borrowed liftimes aslifetime bounds on `'de`.Also like `serde` all top level references automatically have theirlifetimes borrowed, but container types carrying references mustexplicity be borrowed with the `#[postgres(borrow)]` annotation.

restore the cargo.toml in postgres-types

eae83a4

better organization of changes

a7b6a64