[client-v2, jdbc-v2] Lazy String instantiation, Read bytes to buffer first#2813
[client-v2, jdbc-v2] Lazy String instantiation, Read bytes to buffer first#2813chernser wants to merge 9 commits into
Conversation
|
This implementation is bad. Closing PR. |
…ing with typehinting
Client V2 CoverageCoverage Report
Class Coverage
|
JDBC V2 CoverageCoverage Report
Class Coverage
|
|
@claude[agent] review changes. give short summary in comment. |
Review SummaryChanged: This PR adds lazy String instantiation by reading string bytes into ByteBuffers first, with opt-in BinaryString type for deferred conversion. Also includes Session API formalization, improved error messages, Variant null handling, and DNS/auth fixes. Core Changes:
Compatibility Risk: MEDIUM Breaking/High-Risk Changes:
Issues Found:
Tests: Variant null tests added and passing. No dedicated BinaryString tests visible in diff. Session tests added. Docs: No changes to docs/features.md despite new type-hint API being user-facing in custom reader scenarios. |
|
|
This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 2 weeks if no further activity occurs. Please feel free to give a status update or ping for review. Thank you for your contributions! |


Summary
Closes #2263
Closes #2558
Closes #2608
Checklist
Delete items not relevant to your PR: