You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@thrift.apache.org by Apache Wiki <wi...@apache.org> on 2008/08/22 11:23:50 UTC

[Thrift Wiki] Update of "New compact binary protocol" by NoblePaul

Dear Wiki user,

You have subscribed to a wiki page or wiki category on "Thrift Wiki" for change notification.

The following page has been changed by NoblePaul:
http://wiki.apache.org/thrift/New_compact_binary_protocol

------------------------------------------------------------------------------
  
  == Implementation suggestions ==
  === Encode i16, i32 and i64 types saved as variable size integers ===
- ||Suggestion||Pros||Cons||
+ ||'''Suggestion'''||'''Pros'''||'''Cons'''||
+ ||ZIP encoding (variable length encoding) for only positive values|| save a max of 3 bytes for small ints ||user has to specify the new type||
- ||Base 128 + zigzag, borrow from protocol buffers?|| ||User has to specify whether zigzag needs to be used for efficiency||
+ ||Base 128 + zigzag, borrow from protocol buffers?|| ||user has to specify whether zigzag needs to be used for efficiency||
- ''Is this suggestion about variable encoding of all ints, or just specified ones?''
+ 
+ As the user knows best about his data he can choose whichever he wants and save bytes. This means we need more type modifiers for these types
+ 
  
  === Remove / reduce the size of field prefix tags ===
  ||Suggestion||Pros||Cons||