twuebi commented on PR #931: URL: https://github.com/apache/iceberg-go/pull/931#issuecomment-4441415325
Hi @C-Loftus, thanks for the poke, we've fixed the double encoding in arrow-go in the meantime, once arrow-go makes another release, and we bump the dependency here, we should be able to allow dictionary encoding. Until then, enabling it would mean a regression for high cardinality columns. -- 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]
