Closed discord9 closed 1 month ago
The modifications focus on standardizing and anonymizing SQL query patterns within test cases. Specific terms and patterns have been redacted or replaced with placeholders, ensuring consistency and compliance with updated guidelines. These changes improve readability and maintainability without altering the core functionality of the SQL queries.
Files | Change Summary |
---|---|
tests/cases/standalone/optimizer/last_value.sql |
Replaced and redacted specific SQL patterns, including metrics.* , partitioning.* , and others, with placeholders. |
tests/cases/standalone/optimizer/last_value.result |
Adjusted terms, plan structure, and execution steps in the SQL results for clarity and compliance. |
In fields of code where queries grow, Changes came to let patterns flow. With placeholders set and terms refined, Our tests now run with peace of mind. Let the SQL dance, neat and clear, A bunny's joy, we hold dear! 🐰
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?
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 84.88%. Comparing base (
d6c7b84
) to head (baff261
).
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?
ignore flow sqlness tests for now until a
flush_flow
function is added for better test harnessChecklist
Summary by CodeRabbit