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

Fix default values in task properties docs #15089

Merged
merged 1 commit into from
Dec 6, 2022

Conversation

colebow
Copy link
Member

@colebow colebow commented Nov 17, 2022

Description

Putting the docs into plain english to be more user-friendly.

Additional context and related issues

Follow-up to #14553

Release notes

(x) This is not user-visible or docs only and no release notes are required.
( ) Release notes are required, please propose a release note for me.
( ) Release notes are required, with the following suggested text:

@sopel39
Copy link
Member

sopel39 commented Nov 18, 2022

Thx!

Copy link
Member

@mosabua mosabua left a comment

Choose a reason for hiding this comment

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

Looks much better and easier to understand for users now. Can you merge @sopel39 ?

@sopel39 sopel39 merged commit 80afa9b into trinodb:master Dec 6, 2022
@github-actions github-actions bot added this to the 404 milestone Dec 6, 2022
@colebow colebow deleted the colebow/task-concurrency-value branch January 13, 2023 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants