The GitHub Actions job "Fory CI" on fory.git/fory-perf-compact-row-layout has failed. Run started by GitHub user stevenschlansker (triggered by stevenschlansker).
Head commit for run: ff84bb469021949d85dc86361727d398278c3f64 / Claude (on behalf of Steven Schlansker) <[email protected]> perf(format): cache compact row layout per nested slot CompactRowLayout precomputes per-schema fixed offsets, fixed widths, bitmap width, and child layouts, and is shared across all CompactBinaryRow instances built from the same schema. The writer holds the cached layout and exposes it via BaseBinaryRowWriter.newRow(), so each BinaryRowEncoder.decode call allocates a fresh BinaryRow over the already-built layout tree rather than rebuilding it. A fresh row per decode keeps lazy interface-backed decoders safe: those wrap and retain the source BinaryRow, so reusing one row across decode() calls would silently alias every previously returned proxy onto the next payload. The unused Encoding.newRow(Schema) entry point is removed along with the now-dead codecFactory field on BinaryRowEncoder. Also drops the per-inline-struct getBuffer().slice(...) allocation in CompactBinaryRow.getStruct by pointing the nested row directly at the parent buffer, since all nested reads already add baseOffset. Report URL: https://github.com/apache/fory/actions/runs/26731537132 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
