paveon opened a new pull request, #1143:
URL: https://github.com/apache/iceberg-go/pull/1143
## Summary
- Split `dataFile.initializeMapData()` into `initPartitionData()` and
`initColumnStatsData()`, each with its own `sync.Once`, so that
partition-only callers (compaction, conflict validation, snapshots)
no longer allocate column-stats maps they never read.
- Preallocate the map in `avroColMapToMap` with `make(map[K]V, len(*c))`
to avoid incremental rehashing.
Fixes [#1142](https://github.com/apache/iceberg-go/issues/1142)
## Test plan
- Existing tests cover both init paths — column stats are verified in
`manifest_test.go` (v1, v2, round-trip) and partition data in
`table_test.go`, `codec/data_file_test.go`, `dv_writer_test.go`, etc.
- `go test ./...` passes with no regressions.
--
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]
