stevenschlansker opened a new pull request, #3715: URL: https://github.com/apache/fory/pull/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 - [ ] 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 -- 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]
