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.

Sign up for GitHub

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

Jump to bottom

worker-count should has limit to set value #1716

Closed
Tammyxia opened this issue Apr 28, 2021 · 4 comments · Fixed by #4482
Closed

worker-count should has limit to set value #1716

Tammyxia opened this issue Apr 28, 2021 · 4 comments · Fixed by #4482
Assignees
@overvenus
Labels
area/ticdc Issues or PRs related to TiCDC. bug-from-internal-test Bugs found by internal testing. component/cli CLI component. severity/minor This is a minor bug. type/bug This is a bug.
Projects
Question and Bug Reports

Comments

@Tammyxia
Copy link

Bug Report

Please answer these questions before submitting your issue. Thanks!

  1. What did you do? If possible, provide a recipe for reproducing the error.

  • Create changefeed --sink-uri='xxx/worker-count=0'
    image

  • Create changefeed --sink-uri='xxx/worker-count=1000000000000000'
    image

  1. What did you expect to see?
  • worker-count=0 is not allowed to set
  • What is the max value for worker-count? worker-count=1000000000000000 seems too large to allowed be to set.

  1. What did you see instead?
    Successfully to create changefeed with worker-count=0 or worker-count=1000000000000000

  2. Versions of the cluster

    • Upstream TiDB cluster version (execute SELECT tidb_version(); in a MySQL client):

      5.0.0-nightly

    • TiCDC version (execute cdc version):

      5.0.0-nightly
@Tammyxia Tammyxia added type/bug This is a bug. severity/moderate This is a moderate bug. bug-from-internal-test Bugs found by internal testing. labels Apr 28, 2021
@Tammyxia
Copy link
Author

/worker-count=-1 also allowed to create.

@Tammyxia
Copy link
Author

image
too large worker-count makes runtim oom.

@Tammyxia Tammyxia added severity/major This is a major bug. and removed severity/moderate This is a moderate bug. labels Apr 28, 2021
@hi-rustin hi-rustin mentioned this issue Jun 28, 2021
Closed
3 tasks
@overvenus overvenus added this to Need Triage in Question and Bug Reports Jul 3, 2021
@hi-rustin
Copy link
Member

/assign

@hi-rustin
Copy link
Member

/worker-count=-1 also allowed to create.

It will use the default value.

@AkiraXie AkiraXie added the area/ticdc Issues or PRs related to TiCDC. label Dec 6, 2021
@overvenus overvenus added component/cli CLI component. severity/minor This is a minor bug. and removed severity/major This is a major bug. labels Dec 9, 2021
@overvenus overvenus assigned overvenus and unassigned hi-rustin Jan 26, 2022
@overvenus overvenus mentioned this issue Jan 26, 2022
Merged
Question and Bug Reports automation moved this from Need Triage to Done Jan 26, 2022
ti-chi-bot pushed a commit that referenced this issue Jan 26, 2022
@overvenus
cmd,sink(ticdc): validate changefeed params and revise error message (#…
f917d36 
…4482)

close #1716, close #1718, close #1719, close #4472
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@overvenus overvenus

Labels
area/ticdc Issues or PRs related to TiCDC. bug-from-internal-test Bugs found by internal testing. component/cli CLI component. severity/minor This is a minor bug. type/bug This is a bug.
Projects
Question and Bug Reports
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

cmd,sink(ticdc): validate changefeed params and revise error message overvenus/ticdc
4 participants
@overvenus @hi-rustin @AkiraXie @Tammyxia