Closed WenyXu closed 1 week ago
The Display
implementation for the ColumnOption
enum in create_expr.rs
has been updated. Specifically, the branch for ColumnOption::DefaultValue(s)
has been modified to handle Value::String
differently, ensuring the string value is properly formatted with single quotes.
File | Change Summary |
---|---|
tests-fuzz/src/ir/create_expr.rs |
Updated Display implementation for ColumnOption to handle Value::String with specific formatting. |
Amidst the code where values set, A change emerged, we won't forget. Strings framed with quotes, pristine and bright, Making defaults appear just right. 🐇✨ In rustling leaves, the code's delight!
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?
Attention: Patch coverage is 0%
with 4 lines
in your changes missing coverage. Please review.
Project coverage is 84.69%. Comparing base (
6e2c21d
) to head (1e76420
). Report is 1 commits behind head on main.
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?
Enhance ColumnOption::DefaultValue formatting for string values, to fix incorrect CreateTableExpr generation🥲
Checklist
Summary by CodeRabbit