databendlabs / databend

๐——๐—ฎ๐˜๐—ฎ, ๐—”๐—ป๐—ฎ๐—น๐˜†๐˜๐—ถ๐—ฐ๐˜€ & ๐—”๐—œ. Modern alternative to Snowflake. Cost-effective and simple for massive-scale analytics. https://databend.com
https://docs.databend.com
Other
7.87k stars 751 forks source link

fix(query): fix incorrect total_bytes_len in string view #16877

Closed sundy-li closed 23 hours ago

sundy-li commented 2 days ago

I hereby agree to the terms of the CLA available at: https://docs.databend.com/dev/policies/cla/

Summary

  1. add more asserts
  2. fix incorrect total_bytes_len in string view

using const UNKNOWN_LEN: u64 = u64::MAX to track with the total_bytes_len was a bad idea cause it may introduce overflow during add/sub.

closes #16880

Tests

Type of change


This change isโ€‚Reviewable