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

Clarify the meaning of job retries #3028

Draft
wants to merge 4 commits into
base: latest
Choose a base branch
from
Draft

Conversation

jnidzwetzki
Copy link
Contributor

With timescale/timescaledb#6698 the meaning of 0 and -1 retries of a job changes. This PR updates the documentation accordingly.

Description

[Short summary of why you created this PR]

Links

Fixes #[insert issue link, if any]

Writing help

For information about style and word usage, see the style guide

Review checklists

Reviewers: use this section to ensure you have checked everything before approving this PR:

Subject matter expert (SME) review checklist

  • Is the content technically accurate?
  • Is the content complete?
  • Is the content presented in a logical order?
  • Does the content use appropriate names for features and products?
  • Does the content provide relevant links to further information?

Documentation team review checklist

  • Is the content free from typos?
  • Does the content use plain English?
  • Does the content contain clear sections for concepts, tasks, and references?
  • Have any images been uploaded to the correct location, and are resolvable?
  • If the page index was updated, are redirects required
    and have they been implemented?
  • Have you checked the built version of this content?

With timescale/timescaledb#6698 the meaning of 0 and -1 retries of a job
changes. This PR updates the documentation accordingly.
Copy link

Allow 10 minutes from last push for the staging site to build. If the link doesn't work, try using incognito mode instead. For internal reviewers, check web-documentation repo actions for staging build status. Link to build for this PR: http://docs-dev.timescale.com/docs-jnidzwetzki-job-retry

@billy-the-fish
Copy link
Contributor

@jnidzwetzki are you ok if I publish this baby?

@jnidzwetzki
Copy link
Contributor Author

@billy-the-fish The code PR is merged. However, we have not published a new TSDB version containing the code. It will be part of 2.15.0 and I guess we should merge this documentation PR only after the TSDB version is tagged.

@billy-the-fish billy-the-fish marked this pull request as draft March 20, 2024 18:28
@billy-the-fish
Copy link
Contributor

Moved to draft so this is not prematurely published by over-enthusiastic tech writers.

@billy-the-fish
Copy link
Contributor

Hi there @antekresic, can I merge this update?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants