search

lisad / phaser

The missing layer for complex data batch integration pipelines
MIT License
9 stars 1 forks source link

Log correct row number with header consistency check #89

Open jeffkole opened 8 months ago

jeffkole commented 8 months ago

In Phase#check_headers_consistent since the context.current_row is not updated when the warnings are logged, all of the warnings are associated with the last row number, even though the full row (along with its row number) are passed into the add_warning function call.

check_headers_consistent should log which row the error occurred on.

jeffkole commented 7 months ago

Need to revalidate this issue, as it may have been fixed with recent changes.