search

kestra-io / plugin-serdes

https://kestra.io/plugins/plugin-serdes/
Apache License 2.0
2 stars 6 forks source link

feat: upgrade to FastCSV 2.2.2 #79

Closed loicmathieu closed 10 months ago

loicmathieu commented 10 months ago

Fixes #78

This bugfix can be QA with the flow from the issue.

Note that a big refactoring was done due to the major version bump of the CSV library we use, testing multiple task with multiple properties is advise (read, write, with header, without header, ...)

loicmathieu commented 10 months ago

I removed a test on the AvroCOnverterTest that asserts that an incorrect CSV was read but cannot be written in AVRO. Now it generates an exception at CSV read time, which IMHO is better as the CSV file is incorrect.