Skip to content

fix: athena-iceberg/schema-evolution/new-columns-empty-in-athena #3067

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

Closed
wants to merge 10 commits into from
Closed

fix: athena-iceberg/schema-evolution/new-columns-empty-in-athena #3067

Changes from 2 commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
19635ce
bugfix/writing-complete-df
lautarortega Jan 9, 2025
a2e4ed4
Merge branch 'main' into patch-1
jaidisido Jan 13, 2025
d2bbd63
Merge branch 'main' into patch-1
jaidisido Jan 21, 2025
3b847ce
updated implementation
Feb 4, 2025
954e35c
test case added
Feb 4, 2025
3afeb25
Merge branch 'main' into patch-1
lautarortega Feb 4, 2025
3831881
Merge branch 'main' into patch-1
jaidisido Feb 17, 2025
8b1d527
Merge branch 'main' into patch-1
jaidisido Feb 18, 2025
d5dba3a
fix: failing test fix
lautarortega Feb 18, 2025
6e8e88e
Merge branch 'main' into patch-1
kukushking Feb 24, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion awswrangler/athena/_write_iceberg.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -546,7 +546,7 @@ def to_iceberg(

# Ensure that the ordering of the DF is the same as in the catalog.
# This is required for the INSERT command to work.
df = df[catalog_cols]
df = df[catalog_cols + list(schema_differences["new_columns"].keys())]

if schema_evolution is False and any([schema_differences[x] for x in schema_differences]): # type: ignore[literal-required]
raise exceptions.InvalidArgumentValue(f"Schema change detected: {schema_differences}")
Expand Down
Loading