Closed CookiePieWw closed 2 months ago
The recent updates in the codebase introduce a new Binary
variant to the ConcreteDataType
enum. Adjustments have been made to incorporate this new variant in the is_stringifiable
method and include assertions for it in the test module. Additionally, SQL statements are now capable of handling the Binary
variant, including converting double-quoted strings to binary values, with corresponding tests to verify this functionality.
Files | Change Summaries |
---|---|
src/.../data_type.rs |
Added Binary variant to ConcreteDataType . Updated is_stringifiable method and test assertions to include Binary . |
src/sql/.../statements.rs |
Added handling of Binary type in SQL statements, including conversion and tests. |
In the data's humble code, now shines a Binary light,
A new type to stringify, in the quiet of the night. 🌟
SQL's heart beat swifter, with binaries in sight,
Tests bloom like daisies, pure and bright.
The rabbit cheers with joy and might,
For the data now, reads just right.
Hoppity hop, the future is in flight! 🐇✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
I happened to this while working for the tests of PREPARE
stmts, there's a TODO for this, but a little different? I think this TODO means something like b'abcd'
instead of plain string 'abcd'
.
https://github.com/GreptimeTeam/greptimedb/blob/ea7c17089f2ded9ec0db215b42856a700645c287/src/sql/src/statements.rs#L251
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 84.84%. Comparing base (
ea7c170
) to head (54fb3da
).
I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
What's changed and what's your intention?
Introduce a similar behavior from MySQL to GreptimeDB:
Checklist
Summary by CodeRabbit
New Features
Binary
data type.Bug Fixes
Tests
Binary
data type.