Skip to content

Core: Build partition spec with case insensitive source column identifiers#5110

Closed
findinpath wants to merge 1 commit into
apache:masterfrom
findinpath:partition-spec-case-insensitive
Closed

Core: Build partition spec with case insensitive source column identifiers#5110
findinpath wants to merge 1 commit into
apache:masterfrom
findinpath:partition-spec-case-insensitive

Conversation

@findinpath
Copy link
Copy Markdown
Contributor

Address the following scenario:

        onSpark().executeQuery(format(
                "CREATE TABLE %s (id INTEGER, `mIxEd_COL` STRING) USING ICEBERG",
                sparkTableName));
        assertQueryFailure(() -> onTrino().executeQuery("ALTER TABLE " + trinoTableName + " SET PROPERTIES partitioning = ARRAY['mIxEd_COL']"))
                .hasMessageContaining("Cannot find source column: mixed_col);

Trino lowercases internally the identifiers for the column names.

This PR attempts to provide a transparent fashion to allow building a PartitionSpec by specifying the source column names in a case insensitive fashion.

@findinpath findinpath force-pushed the partition-spec-case-insensitive branch from 64b9f43 to cbf1210 Compare June 22, 2022 04:29
@findinpath findinpath closed this Jun 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant