jhorstmann commented on issue #5854:
URL: https://github.com/apache/arrow-rs/issues/5854#issuecomment-2154845559

   Thanks @alamb for creating this tracking issue. I've slowly continued 
working on my code at 
[jhorstmann/compact-thrift](https://github.com/jhorstmann/compact-thrift/) and 
benchmarks are looking good. So good in fact that adapting it on top of #5777 
the performance hotspot shifts to the conversion functions from generated 
thrift types to internal types.
   
   I would love to get some feedback on the code, and whether there would be a 
preference to integrate the parquet definitions into the arrow-rs repo, or 
publish them separately.
   
   The generated and runtime code is also structured in a way that it would not 
be too crazy to write bindings to custom types by hand.
   
   Direct links to [the generated 
code](https://github.com/jhorstmann/compact-thrift/blob/main/src/main/rust/parquet/src/format.rs)
 and to [the runtime 
library](https://github.com/jhorstmann/compact-thrift/blob/main/src/main/rust/runtime/src/lib.rs).


-- 
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]

Reply via email to