iemejia commented on PR #3858: URL: https://github.com/apache/avro/pull/3858#issuecomment-4952729914
**Preallocation (AVRO-4292 follow-up) evaluated — no change needed here.** As part of hardening collection allocation across SDKs (the Java/C++/C# readers were sizing a backing buffer to the declared block count up front), I audited the C path. `read_array_value`/`read_map_value` append one element at a time via `avro_value_append`, with no up-front sizing to the declared block count, so the amortised (grow-on-demand) allocation is already bounded by the elements actually read. No preallocation clamp is required for C. -- 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]
