Skip to content

feat(sql): decode binary output formats - #800

Open
sundy-li wants to merge 1 commit into
mainfrom
feat/reimplement-binary-format
Open

feat(sql): decode binary output formats#800
sundy-li wants to merge 1 commit into
mainfrom
feat/reimplement-binary-format

Conversation

@sundy-li

@sundy-li sundy-li commented Sep 9, 2026

Copy link
Copy Markdown
Member

Summary

  • decode Binary values according to BinaryFormat: HEX, BASE64, UTF-8, and UTF-8-LOSSY
  • apply the configured format to top-level and nested Binary values
  • preserve Map key/value boundaries, including : separators for Binary keys
  • add coverage for top-level formats, nested Base64 Map keys, Hex Map keys, and Nullable Binary
  • replace unsafe UTF-8 conversions in the affected string decoder paths with checked conversions
  • remove the obsolete Hex-only binary token helper

Related

Reimplements the useful parts of closed PR #772.

Verification

  • cargo fmt --all -- --check
  • cargo test -p databend-driver-core --lib
  • cargo test -p databend-driver-core --all-features --lib
  • cargo check -p databend-driver-core
  • cargo clippy -p databend-driver-core --all-targets --all-features -- -D warnings
  • git diff --check

Integration tests requiring a running Databend server were not run.

@sundy-li
sundy-li requested a review from youngsofun September 9, 2026 02:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant