Propagate decode_response to nested structs #194

Merged
merged 1 commit into from Mar 30, 2016

Conversation

Projects
None yet
2 participants
@wbolster
Contributor

wbolster commented Mar 28, 2016

Nested maps, lists, sets and structs now correctly take the
decode_response argument into account instead of unconditionally
attempting bytes to text decoding.

Fixes #193.

Propagate decode_response to nested structs
Nested maps, lists, sets and structs now correctly take the
"decode_response" argument into account instead of unconditionally
attempting bytes to text decoding.

Fixes #193.
@wbolster

This comment has been minimized.

Show comment
Hide comment
@wbolster

wbolster Mar 28, 2016

Contributor

Note that the compact protocol implementation only uses calls to self.read_...() which uses the self.decode_response value correctly in all cases.

Contributor

wbolster commented Mar 28, 2016

Note that the compact protocol implementation only uses calls to self.read_...() which uses the self.decode_response value correctly in all cases.

@lxyu lxyu merged commit d9afc32 into eleme:develop Mar 30, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@lxyu

This comment has been minimized.

Show comment
Hide comment
@lxyu

lxyu Mar 30, 2016

Contributor

Thanks very much!

Contributor

lxyu commented Mar 30, 2016

Thanks very much!

@wbolster wbolster referenced this pull request in wbolster/happybase Apr 29, 2016

Closed

How to disable thriftpy's auto decoding? #122

@pyup-bot pyup-bot referenced this pull request in scieloorg/opac_proc Aug 18, 2016

Closed

Initial Update #10

@pyup-bot pyup-bot referenced this pull request in scieloorg/scielo-manager Aug 26, 2016

Closed

Pin thriftpy to latest version 0.3.9 #1314

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment