druid icon indicating copy to clipboard operation
druid copied to clipboard

Allow non literal rhs in MV_FILTER_ONLY and MV_FILTER_NONE

Open sreemanamala opened this issue 1 year ago • 0 comments

Description

This PR allows to use the MV_FILTER_ONLY & MV_FILTER_NONE functions with a non literal argument. Currently select mv_filter_only('mvd_dim', 'array_dim') from 'table' returns a Unhandled Query Planning Failure This is being tackled and also considered for the cases where the array_dim having null & empty values


Key changed/added classes in this PR
  • MultiValueStringOperatorConversions
  • ApplyFunction
  • CalciteMultiValueStringQueryTest

This PR has:

  • [x] been self-reviewed.
    • [ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)
  • [ ] added documentation for new or modified features or behaviors.
  • [ ] a release note entry in the PR description.
  • [ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
  • [ ] added or updated version, license, or notice information in licenses.yaml
  • [ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • [x] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
  • [ ] added integration tests.
  • [ ] been tested in a test Druid cluster.

sreemanamala avatar Mar 13 '24 02:03 sreemanamala