You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When merging data into a Delta table using DeltaTableWriter with the merge_builder param one needs to provide the source DataFrame and the target table name as arguments to DeltaTableWriter, even though they are already passed to the merge builder. Otherwise Koheesio throws an error about missing required fields. So effectively we are passing the same values twice.
Describe the bug
Take this example of merging data into a Delta table using merge_builder:
Even though in the merge_builder param we already pass source_df and target_table_name, we still need to pass the same values as the df and table arguments respectively to DeltaTableWriter. Otherwise Pydantic will throw a Validation Error.
Steps to Reproduce
Use the above code snippet to merge data into a Delta table using merge builder to trigger the error.
Expected behavior
When using a merge builder with appropriate params, one should not be forced to additionally pass the df and table params to DeltaTableWriter.
Environment
Koheesio 0.9.0
The text was updated successfully, but these errors were encountered:
When merging data into a Delta table using
DeltaTableWriter
with themerge_builder
param one needs to provide the source DataFrame and the target table name as arguments toDeltaTableWriter
, even though they are already passed to the merge builder. Otherwise Koheesio throws an error about missing required fields. So effectively we are passing the same values twice.Describe the bug
Take this example of merging data into a Delta table using merge_builder:
Even though in the
merge_builder
param we already passsource_df
andtarget_table_name
, we still need to pass the same values as thedf
andtable
arguments respectively toDeltaTableWriter
. Otherwise Pydantic will throw a Validation Error.Steps to Reproduce
Use the above code snippet to merge data into a Delta table using merge builder to trigger the error.
Expected behavior
When using a merge builder with appropriate params, one should not be forced to additionally pass the
df
andtable
params toDeltaTableWriter
.Environment
Koheesio 0.9.0
The text was updated successfully, but these errors were encountered: