You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/11/30 07:33:00 UTC

[GitHub] [arrow-datafusion] retikulum commented on issue #4386: Make Binary Dictionary Operations Optional

retikulum commented on issue #4386:
URL: https://github.com/apache/arrow-datafusion/issues/4386#issuecomment-1331748181

   Hi. I added this on purpose (but without knowing it is extremely expensive) to pass `test_dictionary_type_to_array_coersion` test case. The following error was generated before enabling it:
   ```rust 
   Error: ArrowError(CastError("Comparing array of type Dictionary(Int32, Utf8) with array of type Dictionary(Int32, Utf8) requires \"dyn_cmp_dict\" feature"))
   ```
   It seems great to me for enabling the feature flag rather than enabling it by default. Thanks for noticing me.


-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org