-
Notifications
You must be signed in to change notification settings - Fork 1
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
Added ability to schedule sql jobs #483
Conversation
7cd01eb
to
e059a1a
Compare
docs/commands/gcjobs.rst
Outdated
``gc`` | ||
====== | ||
|
||
The ``gc`` command lets you manage scheduled sql jobs for your cluster. |
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.
I'm wondering if the keyword here should be something likescheduled-sql
instead of gc
. I guess we could have other GC features in croud later, it might be confusing. idk.
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.
I think I agree here, gc is not really a public facing thing...
Maybe croud scheduled-jobs
?
my_table.csv
Outdated
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.
Is it used in the tests? I can't find it.
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.
thanks for finding this. It appears that when you run the whole test suit, the test test_export_job_create
writes the my_table.csv
file but does not clean it up ...
cc7c5af
to
7e380f5
Compare
Summary of the changes / Why this is an improvement
Checklist