Skip to content
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

Add next_start option to alter_job_schedule #1479

Merged
merged 1 commit into from Oct 11, 2019

Conversation

cevian
Copy link
Contributor

@cevian cevian commented Oct 11, 2019

Add the option to set the next start time on a job in the
alter job schedule function. This also adds the ability
to pause jobs by setting next_start to 'infinity'

Also fix the enterprise licence check to only activate for
enterprise jobs.

@cevian cevian force-pushed the add_next_start_to_alter_schedule branch from 30e0f30 to 5b43f58 Compare October 11, 2019 15:46
@codecov
Copy link

codecov bot commented Oct 11, 2019

Codecov Report

Merging #1479 into master will decrease coverage by 8.6%.
The diff coverage is 90.47%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #1479      +/-   ##
=========================================
- Coverage      90%   81.4%   -8.61%     
=========================================
  Files         116     116              
  Lines       17495   16934     -561     
=========================================
- Hits        15747   13785    -1962     
- Misses       1748    3149    +1401
Flag Coverage Δ
#cron 81.4% <90.47%> (-8.61%) ⬇️
#pr 81.4% <90.47%> (+0.08%) ⬆️
Impacted Files Coverage Δ
tsl/src/bgw_policy/job.c 88.96% <100%> (-4.19%) ⬇️
src/bgw/job.c 94.48% <100%> (-0.12%) ⬇️
src/bgw/job_stat.c 94.4% <86.66%> (-2.28%) ⬇️
tsl/src/nodes/gapfill/gapfill.c 45.45% <0%> (-54.55%) ⬇️
src/chunk_insert_state.c 31.62% <0%> (-53.95%) ⬇️
src/chunk.h 0% <0%> (-50%) ⬇️
src/utils.h 0% <0%> (-50%) ⬇️
src/plan_add_hashagg.c 46.87% <0%> (-40.63%) ⬇️
src/plan_expand_hypertable.c 43.28% <0%> (-40.32%) ⬇️
src/copy.c 44.67% <0%> (-35.98%) ⬇️
... and 86 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 01f2bba...e0cc70e. Read the comment docs.

Copy link
Contributor

@davidkohn88 davidkohn88 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine to me, but I think it'd be good if Josh looked at it briefly.

\c :TEST_DBNAME :ROLE_DEFAULT_PERM_USER
--not changing the interval doesn't change the next_start
select * from alter_job_schedule(:job_id, schedule_interval=>'20 min');
WARNING: Timescale License expired
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we probably shouldn't check license expiration on calls of this function if we've taken it out of enterprise

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

but not sure if that's true/if we do it elsewhere...

Add the option to set the next start time on a job in the
alter job schedule function. This also adds the ability
to pause jobs by setting next_start to 'infinity'

Also fix the enterprise licence check to only activate for
enterprise jobs.
@cevian cevian force-pushed the add_next_start_to_alter_schedule branch from 5b43f58 to e0cc70e Compare October 11, 2019 18:19
@cevian cevian merged commit 2209133 into timescale:master Oct 11, 2019
@cevian cevian added this to the 1.5.0 milestone Oct 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants