Closed discord9 closed 3 weeks ago
Attention: Patch coverage is 92.46704%
with 40 lines
in your changes missing coverage. Please review.
Project coverage is 84.81%. Comparing base (
5566dd7
) to head (fa89509
).
[!WARNING]
Rate limit exceeded
@discord9 has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 10 minutes and 19 seconds before requesting another review.
How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://coderabbit.ai/docs/faq) for further information.Commits
Files that changed from the base of the PR and between 768ffcf053d552e8b0bece8676a72ebbfbb407a1 and fa89509b7966b5f9a86f4440568f44eadeec777e.
Recent changes enhance the FlownodeManager
by standardizing column names to lowercase and improve the TableSource
with a new auto_columns
feature. Significant updates in WorkerHandle
handling and test optimizations, as well as enriched expressions in scalar.rs
, relation.rs
, and plan.rs
files, bolster functionality and testing. Additionally, SQL operations were added to the tests, and the sqlness
dependency was updated. These refinements collectively promote consistency, error handling, and code robustness.
File(s) | Summary |
---|---|
src/.../adapter.rs |
Modified column name formatting to lowercase in FlownodeManager . |
src/.../adapter/table_source.rs |
Introduced auto_columns to signify non-auto columns in TableSource . |
src/.../adapter/worker.rs |
Updated drop method to use block_on for WorkerHandle and adjusted tests accordingly. |
src/.../expr/scalar.rs |
Enhanced support for expressions and evaluation functions, including comment updates and new methods. |
src/.../repr/relation.rs |
Added auto_columns to RelationType and adjusted RelationDesc with error handling for field length mismatch. |
src/.../transform/aggr.rs |
Added logic for marking auto-added columns and extended DataFrame function tests. |
src/.../transform/expr.rs |
Introduced proto_col function and refactored existing functions to improve expression handling. |
src/.../transform/plan.rs |
Refined from_substrait_rel method and added methods to handle projects, filters, and reads. |
tests/cases/.../flow/basic.result |
Added SQL operations for table creation, data manipulation, flow definition, and querying. |
tests/cases/.../flow/df_func.result |
Added SQL operations for tables and flows using DataFrame functions and absolute sums. |
tests/runner/Cargo.toml |
Updated sqlness dependency version from "0.5" to "0.6.1" . |
A bunny hopped through fields of code, With agile paws, it lightened the load. From capital columns, lowercase leapt, Auto columns marked, promises kept. Dependencies grew, tighter they bound, In SQL flows, new strengths were found. πβ¨π»
This summary encapsulates the file-specific changes and includes a whimsical poem to celebrate the improvements.
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 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?
fix a bug when calling df func like
abs
while nested some flow's own function likeabs(sum(number))
, and also added some sqlness testsPlease explain IN DETAIL what the changes are in this PR and why they are needed:
abs(sum(number))
Checklist
Summary by CodeRabbit
New Features
Bug Fixes
Improvements
WorkerHandle
shutdown process with better error handling and logging.auto_columns
to signify non-auto columns in table schemas.Dependency Updates
sqlness
dependency version from0.5
to0.6.1
.