badalprasadsingh commented on PR #524: URL: https://github.com/apache/iceberg-go/pull/524#issuecomment-3316310850
Hey @zeroshade, I have updated the workflow. While writing a manifest, we need to encode it into Avro types. So, just before encoding in the `addEntry()` function, we call `avroPartitionData()`, which converts the data into the respective maps. While reading manifests, the decoded Avro types come as Go types, and we use the `convertAvroValueToIcebergType` function inside `initializeMapData()` to convert them back into the respective Iceberg literals. The `getFieldIDMap()` func is used both during writing manifests and during reading manifests to get the `fieldNameToID`, `fieldIDToType`, `fieldIDToSize` (only during reading) maps. Thus, we need to have `int` type as a case as well (during writing). -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
