Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement text array decoder #60

Merged
merged 2 commits into from
Jan 4, 2017
Merged

Implement text array decoder #60

merged 2 commits into from
Jan 4, 2017

Conversation

elprans
Copy link
Member

@elprans elprans commented Jan 3, 2017

No description provided.

@elprans elprans requested a review from 1st1 January 3, 2017 15:32
@elprans elprans force-pushed the textarray branch 2 times, most recently from 18139a1 to 86f17ce Compare January 3, 2017 17:13
Current asyncpg codec architecture allows for only one codec per data
type.  This arrangement works most of the time, but fails utterly to
handle situations when the text format I/O is forced by the lack of
binary I/O support for one of the types in the composite type.

This commit is the preparation for the introduction of text format
parsing and allows to associate multiple format-specific codecs for each
type.
Asyncpg now supports decoding arrays of types that do not support
binary I/O.
Copy link
Member

@1st1 1st1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@elprans elprans merged commit 72ecbf3 into master Jan 4, 2017
@elprans elprans deleted the textarray branch January 5, 2017 00:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants