The GitHub Actions job "Fory CI" on fory.git/main has succeeded.
Run started by GitHub user chaokunyang (triggered by chaokunyang).

Head commit for run:
598e82f5fac0fdfb3871659a5646e0be82f660cc / Steven Schlansker 
<[email protected]>
fix(format): pass row body size, not full payload size, to BinaryRow.pointTo 
(#3715)

## Why?

Minor fixes
- Latent bug in BinaryRowEncoder exposes 8 bytes beyond end of intended
buffer to BinaryRow
- Add reminder documentation for byte[] encoder forms why embedded size
is ignored. Tripped up both me and Claude
- Clean up docstring. Identified during AI review of forthcoming feature
branches

## AI Contribution Checklist

- [x] Substantial AI assistance was used in this PR: `yes` / `no`

AI Usage Disclosure
- substantial_ai_assistance: yes
- scope: all
- affected_files_or_subsystems: Java row format
- ai_review: :heavy_check_mark: 
- ai_review_artifacts: 

> ● Done. Three commits amended on fory-docs-encoders-cleanup: tightened
comments, tightened test, tightened commit messages, no functional
changes since the previous fix was already correct. Force-push when
you're ready.

- human_verification: :heavy_check_mark: 
- performance_verification: N/A
- provenance_license_confirmation: :heavy_check_mark: 

## Does this PR introduce any user-facing change?

No

---------

Co-authored-by: Claude (on behalf of Steven Schlansker) 
<[email protected]>

Report URL: https://github.com/apache/fory/actions/runs/26711198374

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to