Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix!: EXPOSED-288 Extend ANY and ALL operators to use ArrayColumnType #1992

Merged
merged 6 commits into from
Feb 20, 2024
Merged

fix!: EXPOSED-288 Extend ANY and ALL operators to use ArrayColumnType #1992

merged 6 commits into from
Feb 20, 2024

Commits on Feb 20, 2024

  1. feat: EXPOSED-288 Extend ANY and ALL operators to use ArrayColumnType 

    - Support using ANY and ALL operators with array column or expression.
- Remove UntypedAndUnsizedArrayColumnType and replace it in AllAnyFromArrayOp
with ArrayColumnType. Remove associated unused dialect data types.
- Introduce function that resolves a column type based on type reflection. Refactor
existing array and json functions to use this, while still having a nullable override.
    @bog-walk
    bog-walk committed Feb 20, 2024
    Configuration menu
    Copy the full SHA
    2c6bdbc View commit details
    Browse the repository at this point in the history

  2. feat: EXPOSED-288 Extend ANY and ALL operators to use ArrayColumnType 

    Fix logic in arrayParam() and arrayLiteral().

Switch tests to use type reflection and update KDocs about nullable arrays.
    @bog-walk
    bog-walk committed Feb 20, 2024
    Configuration menu
    Copy the full SHA
    e0c4dd8 View commit details
    Browse the repository at this point in the history

  3. fix!: EXPOSED-288 Extend ANY and ALL operators to use ArrayColumnType 

    - Extract unrelated non-breaking changes to another PR
- Add missing Exposed column types to resolve and adjust tests
    @bog-walk
    bog-walk committed Feb 20, 2024
    Configuration menu
    Copy the full SHA
    fdc0f96 View commit details
    Browse the repository at this point in the history

  4. fix!: EXPOSED-288 Extend ANY and ALL operators to use ArrayColumnType 

    - Fix rebase inconsistency
- Update unit test to include arrayParam
- Refactor AllAnyFromExpressionOp to not perform resolve column logic
    @bog-walk
    bog-walk committed Feb 20, 2024
    Configuration menu
    Copy the full SHA
    fb609ca View commit details
    Browse the repository at this point in the history

  5. fix!: EXPOSED-288 Extend ANY and ALL operators to use ArrayColumnType 

    Fix detekt issue
    @bog-walk
    bog-walk committed Feb 20, 2024
    Configuration menu
    Copy the full SHA
    b108f97 View commit details
    Browse the repository at this point in the history

  6. fix!: EXPOSED-288 Extend ANY and ALL operators to use ArrayColumnType 

    Introduce InternalApi annotation and add it to resolveColumnType().
Move other existing annotation classes to dedicated file.
    @bog-walk
    bog-walk committed Feb 20, 2024
    Configuration menu
    Copy the full SHA
    af208a8 View commit details
    Browse the repository at this point in the history