fallintoplace opened a new pull request, #2362: URL: https://github.com/apache/iceberg-go/pull/2362
## What Reuse schema history while decoding partition plans. ## Why Looking up each historical source field clones the schema history again. ## Implementation - Load the current schema and history once per plan, only when needed. - Keep current-schema priority and historical lookup order. ## Benchmark Source-type lookup only, with defensive schema copies. Apple M1 Pro, Go 1.26.3, median of 3 runs: - **16 schemas, 4 historical sources:** 605 µs → 206 µs (**2.9x faster**). - **64 schemas, 16 historical sources:** 19.09 ms → 1.70 ms (**11.2x faster**). `GOMAXPROCS=1 go test ./catalog/rest -run '^$' -bench '^BenchmarkPartitionSourceTypeSchemaHistory$' -benchmem -benchtime=100ms -count=3 -cpu=1` -- 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]
