allthingssecurity commented on code in PR #28030: URL: https://github.com/apache/camel/pull/28030#discussion_r4306582256
########## 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: Thanks, applied as suggested in 970362e118e4. The upgrade-guide entry now says that fixed-length files lost their first row too, that `ignoreFirstRecord` only applies to unmarshal after this change, and that routes which set `ignoreFirstRecord=false` to work around it are not affected. _Claude Code on behalf of allthingssecurity_ -- 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]
