etseidl commented on issue #7909: URL: https://github.com/apache/arrow-rs/issues/7909#issuecomment-3140426438
Thank you @jhorstmann. > On the other hand, the macro approach allows to easily replace one of the types with a custom implementation. For example you could do `impl CompactThriftProtocol for ParquetSchema` and then replace > > ``` > 2: required list<SchemaElement> schema; > ``` > > with > > ``` > 2: required ParquetSchema schema; > ``` > > Other fields that might benefit from custom implementations are statistics, size_statistics and encoding_stats. This is nice to know. It would allow quickly getting a new framework in place and then slowly customizing to improve performance. Once I've finished my experiment with row groups, I'll definitely want to circle back to this for the real implementation! -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
