Skip to content

Fix Parquet table import -u to skip missing table columns and match CSV warning behavior#10611

Merged
elianddb merged 1 commit intomainfrom
elian/10589
Mar 3, 2026
Merged

Fix Parquet table import -u to skip missing table columns and match CSV warning behavior#10611
elianddb merged 1 commit intomainfrom
elian/10589

Conversation

@elianddb
Copy link
Copy Markdown
Contributor

@elianddb elianddb commented Mar 2, 2026

Fix #10589

  • Parquet dolt table import -u no longer fails immediately when the destination table has additional columns missing from the file.
  • Missing destination columns now produce the standard schema mismatch warning path for Parquet.

@coffeegoddd
Copy link
Copy Markdown
Contributor

@elianddb DOLT

comparing_percentages
100.000000 to 100.000000
version result total
86a1bd4 ok 5937471
version total_tests
86a1bd4 5937471
correctness_percentage
100.0

@coffeegoddd
Copy link
Copy Markdown
Contributor

@elianddb DOLT

test_name from_latency_p95 to_latency_p95 percent_change
tpcc-scale-factor-1 61.08 61.08 0.0
test_name from_server_name from_server_version from_tps to_server_name to_server_version to_tps percent_change
tpcc-scale-factor-1 dolt 1912c5a 38.06 dolt 86a1bd4 37.59 -1.23

@elianddb elianddb marked this pull request as ready for review March 2, 2026 23:55
Copy link
Copy Markdown
Contributor

@jycor jycor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@elianddb elianddb merged commit 8b3e5ff into main Mar 3, 2026
45 of 50 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Inconsistent table import behaviour between CSV and Parquet when columns missing in flat file

3 participants