validate field types when write data to transport #149

Merged
merged 1 commit into from Aug 5, 2015

Conversation

Projects
None yet
5 participants
@maralla
Contributor

maralla commented Aug 5, 2015

When type mismatched this kind of exception will be rasied:

TDecodeException: Field 'amount(1)' of 'TGift' needs type 'MAP<STRING, TFood>', but the value is `'hello'`

@lxyu

@lxyu

This comment has been minimized.

Show comment
Hide comment
@lxyu

lxyu Aug 5, 2015

Contributor

👍

Contributor

lxyu commented Aug 5, 2015

👍

lxyu added a commit that referenced this pull request Aug 5, 2015

Merge pull request #149 from maralla/validate_field_types
validate field types when write data to transport

@lxyu lxyu merged commit 32f9696 into eleme:develop Aug 5, 2015

1 check passed

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

This comment has been minimized.

Show comment
Hide comment
@kols

kols Aug 5, 2015

这个 feature 好!

kols commented Aug 5, 2015

这个 feature 好!

@hit9

This comment has been minimized.

Show comment
Hide comment
@hit9

hit9 Aug 5, 2015

Contributor

👍

Contributor

hit9 commented Aug 5, 2015

👍

@wooparadog

This comment has been minimized.

Show comment
Hide comment
@wooparadog

wooparadog Aug 6, 2015

Contributor

其实 python ducktyping 最好是在 throw 的时候处理 error msg 的.

Contributor

wooparadog commented Aug 6, 2015

其实 python ducktyping 最好是在 throw 的时候处理 error msg 的.

@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