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

extend support to idl containers #9

Open
gregory opened this issue Jan 11, 2016 · 2 comments
Open

extend support to idl containers #9

gregory opened this issue Jan 11, 2016 · 2 comments

Comments

@gregory
Copy link

gregory commented Jan 11, 2016

doc: https://thrift.apache.org/docs/types#containers

Would be nice to get list, setand map

@coopernurse
Copy link
Owner

Hi there. Lists are supported already using []. Maps would be possible assuming the keys are always strings (as required by JSON). How would you expect sets to work?

@gregory
Copy link
Author

gregory commented Jan 14, 2016

hey! True, i got away with the [] and struct for map. Not sure about set, but i was wondering why and how hard would it be to implement those idl types?

ps: i started to work on a barrister-amqp FYI: https://github.com/gregory/barrister-amqp

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants