-
Notifications
You must be signed in to change notification settings - Fork 787
feat: impl Task for private #18311
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
feat: impl Task for private #18311
Conversation
🤖 Smart Auto-retry Analysis (Retry 13)
📊 Summary
❌ NO RETRY NEEDEDAll failures appear to be code/test issues requiring manual fixes. 🔍 Job Details
🤖 AboutAutomated analysis using job annotations to distinguish infrastructure issues (auto-retried) from code/test issues (manual fixes needed). |
None => None, | ||
Some(ref w) => Some(mt::WarehouseOptions { | ||
warehouse: w.warehouse.clone(), | ||
using_warehouse_size: w.using_warehouse_size.clone(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why need warehouse_size?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consistent with the current task structure defined by cloud, warehouse_options is rarely used in private
I hereby agree to the terms of the CLA available at: https://docs.databend.com/dev/policies/cla/
Summary
Currently, Task-related functions rely on Cloud. This pr is used to support Task functions when privately deployed.
TODO:
task_history
Tests
Type of change
This change is