Tzu Gwo
Tzu Gwo
as you said, it is required by thrift spec to provide the backwards-compatibility of IDL
Thank you, please fix the CI error
Ok, I will try to fix CI or check it manually, this looks good to me.
Please give more informations to reproduce it.
Thriftpy was not supported any more, and it already be resolved in [thriftpy2](https://github.com/Thriftpy/thriftpy2), thanks!
I think it would be better to keep backward compatibility, I have to release a minor version after this pull request be merged, it is not necessary
Could you please post your thrift IDL file here? I will try to reproduce it.
Well, I tried TJSONProtocol of thriftpy just now, it seems incompatible with Apache Thrift. The request of method you posted above would be serialized to `[1,"doFoo",1,0,{"1":{"rec":{"1":{"str":"1"},"2":{"str":"1"},"3":{"str":"sdf"},"4":{"str":"adsf"}}}}]`(uses `TJSONProtocol`) or `{"request":{"uuid":"1","url":"1","user":"sdf","password":"adsf"}}`(uses `TSimpleJSONProtocol`)...
Use TBinaryProtocol rather than TJSONProtocol, if you want to use both Thriftpy and Apache Thrift at the same time.