[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#2813
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. |
|


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