alamb commented on PR #5545: URL: https://github.com/apache/arrow-datafusion/pull/5545#issuecomment-1466623543
> You get a double-encoded dictionary? I don't know, but IMHO data_type+dictionary_index_type screams "just use DataType instead of inventing yet another type" Yes I agree -- that is what probably should have been done in the first place, but it was not. So now we need to figure out a reasonable way to help avoid subtle regressions. Maybe we can handle it by updating the docs / making it easy to do the right thing (use dictionary) 🤔 -- 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]
