Skip to content

feat: hide sensitive columns in SnowflakeCompare#511

Draft
sh13m wants to merge 6 commits intocapitalone:developfrom
sh13m:hide-cols-snowpark
Draft

feat: hide sensitive columns in SnowflakeCompare#511
sh13m wants to merge 6 commits intocapitalone:developfrom
sh13m:hide-cols-snowpark

Conversation

@sh13m
Copy link
Copy Markdown

@sh13m sh13m commented Apr 15, 2026

Ported over sensitive column hiding to SnowflakeCompare.

  • Moved some methods over into BaseCompare to be synced with PolarsCompare PR (possibly subject to change).
  • Added unit tests.
  • cast_column_names_lower is not a supported class attribute, and instead the dataframe column names are cast to uppercase to be case-insensitive in _validate_dataframe.
    • sensitive columns setting and validation intent is modified to match this behaviour along with the unit tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant