thriftpy icon indicating copy to clipboard operation
thriftpy copied to clipboard

Optionally use __slots__ for payload members

Open misakwa opened this issue 8 years ago • 4 comments

Fixes #198

  • Add TSPayload for slot based structs
  • Add new optional use_slots parameter to allow generating code with slots
  • Cache slot based objects slightly differently to avoid cache poisoning

misakwa avatar Nov 18 '16 14:11 misakwa

There are still a few questions and some cleanup to be done, but I think this should get the conversation started. /cc @dan-blanchard

misakwa avatar Nov 18 '16 16:11 misakwa

Feature accepted. Update this issue later.

hit9 avatar Nov 21 '16 03:11 hit9

I have updated the implementation to support all types of objects as well as the recursive case. The service args and results are now created with slots as well.

misakwa avatar Nov 22 '16 05:11 misakwa

This is now ready for review.

misakwa avatar Nov 22 '16 13:11 misakwa