davsclaus commented on code in PR #28030: URL: https://github.com/apache/camel/pull/28030#discussion_r4306089330
########## docs/user-manual/modules/ROOT/pages/camel-4x-upgrade-guide-4_24.adoc: ########## @@ -743,6 +743,12 @@ now read as a local date-time without a zone (`getDate()` returns a `LocalDateTi definition of an earlier message, if there was one. A calendar that defines a `TZID` an earlier message has defined differently is now read with its own definition. +=== camel-flatpack - marshal writes the first row of the list + +With the default `ignoreFirstRecord=true`, marshalling a `List<Map<String, Object>>` skipped the first map, so the +first data row was missing from the output (the header line is written from the column names). Every map of the list +is now written. A route that added a dummy first map to work around this now gets it in the output too. Review Comment: Could this also mention fixed-length (no header line there, and the first map was skipped too), and say that routes which set `ignoreFirstRecord=false` to work around this are not affected? After this change the option only applies to unmarshal. ```suggestion With the default `ignoreFirstRecord=true`, marshalling a `List<Map<String, Object>>` skipped the first map, so the first data row was missing from the output, for delimited and fixed-length files alike (the header line of a delimited file is written from the column names, not from the first map). Every map of the list is now written, and `ignoreFirstRecord` only applies to unmarshal. A route that set `ignoreFirstRecord=false` to work around this is not affected; a route that added a dummy first map now gets it in the output too. ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
