fallintoplace opened a new pull request, #2244: URL: https://github.com/apache/iceberg-go/pull/2244
### What Build normalized Variant paths without creating temporary strings for each member. ### Why Path normalization can process many members, and temporary strings add work and allocations. ### Implementation - Write the normalized path into a single builder. - Keep the existing escaping rules for nested and unusual members. ### Benchmark - Go 1.26.3, Apple M1 Pro, median of 5 runs. - Escaped path: 562.1 → 193.4 ns/op (about 2.9x), 296 → 48 B/op, 12 → 1 allocations. - Nested 16-member path: 739.9 → 552.8 ns/op (about 1.3x), 6 → 1 allocations. - These benchmarks measure `NormalizeVariantPath` directly. -- 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]
