Skip to content
This repository has been archived by the owner on Dec 10, 2018. It is now read-only.

Parser doesn't handle circular dependencies #280

Open
sicklydove opened this issue Feb 22, 2017 · 2 comments
Open

Parser doesn't handle circular dependencies #280

sicklydove opened this issue Feb 22, 2017 · 2 comments

Comments

@sicklydove
Copy link

sicklydove commented Feb 22, 2017

Thrift includes support for circular dependencies - although the python thrift generation doesn't yet support this feature. When trying to load thrift files with cyclic dependencies, the parser bombs out with a thriftpy.parser.exc.ThriftParserError e.g:

struct TFoo {
  1. required string a_string, 
  2: optional TBar bar,
}

struct TBar {
  1: required i32 some_id,
  2: required TFoo foo,
}
@ethe
Copy link
Member

ethe commented Sep 26, 2018

It's a known defect, thanks for reporting, I will try to fix it in the future.

@ethe
Copy link
Member

ethe commented Dec 9, 2018

I tried to fix it in thriftpy2, go to Thriftpy/thriftpy2#21 and get more informations, hope to see your feedback, thanks.

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

No branches or pull requests

2 participants