Closed discord9 closed 2 months ago
The modifications across the SQL files primarily adjust the sleep durations in various SQL operations, reducing them from 3 seconds to 2 seconds, and introduce additional 500 milliseconds sleep commands. These changes are aimed at optimizing the timing and execution flow of the SQL scripts.
Files/Groups | Change Summary |
---|---|
tests/cases/standalone/common/flow/basic.sql tests/cases/standalone/common/flow/df_func.sql tests/cases/standalone/common/flow/basic.result tests/cases/standalone/common/flow/df_func.result |
Adjusted sleep durations from 3 seconds to 2 seconds. Added new 500 milliseconds sleep commands in various SQL queries. |
In SQL scripts, the sleep did sway,
From three to two, it found its way,
A half-second pause, a gentle nudge,
To speed things up, without a grudge.
The tables turn, with time refined,
A bunny's touch, so well-designed. 🐇💻
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?
@discord9 I have updated the title to fix the semantic check.
https://github.com/GreptimeTeam/greptimedb/actions/runs/9949757194/job/27486637312?pr=4373 The sqlness test still fails.
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 84.87%. Comparing base (
2b912d9
) to head (4e81a34
). Report is 4 commits behind head on main.
There is also another issue related to flow sqlness test. https://github.com/GreptimeTeam/greptimedb/issues/4372
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?
add more sleep intrinsic in flow's sqlness test, so to make it more serial and prevent getting wrong result by preventing query to happen out of order
Checklist
Summary by CodeRabbit