stevenschlansker opened a new issue, #3117:
URL: https://github.com/apache/fory/issues/3117

   ### Search before asking
   
   - [x] I had searched in the [issues](https://github.com/apache/fory/issues) 
and found no similar issues.
   
   
   ### Version
   
   In the original version of Compact Codec, I made sure to in-line fixed width 
binary structures, so that there is no overhead of the indirection of a packed 
size when it is always known. This is a key optimization of the Compact codec.
   
   In #2931, this optimization was eliminated.
   
   I agree with Fory having its own type system instead of importing Arrow 
types, but the fixed width binary is a key optimization for our use case that 
uses many UUID objects. Is it possible to bring this functionality back?
   
   ### Component(s)
   
   Java
   
   ### Minimal reproduce step
   
   Test case:
   
   
https://github.com/apache/fory/commit/f3d46295a1de0b3d56e2c4820f8631dae4c0065c#diff-a9a719017351d352dffa11e97e9bc0cbfb4a74b56766eeb6fde5021ffe17c684R184-R193
   
   ### What did you expect to see?
   
   The original size serializes the test case to 17 bytes
   
   ### What did you see instead?
   
   Now, the encoded size is 32 bytes, almost a 100% increase.
   
   ### Anything Else?
   
   _No response_
   
   ### Are you willing to submit a PR?
   
   - [ ] I'm willing to submit a PR!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to