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

[Bug][Sort-Standalone] NP when init ClsSink and data race problem when first get ClsIdConfig field list #3332

Closed
2 tasks done
vernedeng opened this issue Mar 23, 2022 · 0 comments · Fixed by #3333
Closed
2 tasks done

[Bug][Sort-Standalone] NP when init ClsSink and data race problem when first get ClsIdConfig field list #3332

vernedeng opened this issue Mar 23, 2022 · 0 comments · Fixed by #3333
Assignees
@vernedeng
Labels
type/bug Something is wrong
Milestone
1.1.0

Comments

@vernedeng
Copy link
Contributor

What happened

  1. NP when init deletingClients and ClsWorkers
  2. data race in ClsIdConfig, result in unexpected keyList

What you expected to happen

  1. init deletingClients and ClsWorkers susccessfully
  2. only one thread can produce keyList when first time invoke ClsIdConfig::getFieldList

How to reproduce

Run Sort-Standalone cls sink

Environment

No response

InLong version

master

InLong Component

Other for not specified component

Are you willing to submit PR?

  • Yes, I am willing to submit a PR!

Code of Conduct
@vernedeng vernedeng added the type/bug Something is wrong label Mar 23, 2022
@vernedeng vernedeng changed the title [Bug][Sort-Standalone] Fix NP and data race problem in CLS sink [Bug][Sort-Standalone] NP when init ClsSink and data race problem when first get ClsIdConfig field list Mar 23, 2022
@vernedeng vernedeng mentioned this issue Mar 23, 2022
Merged
1 task
@healchow healchow added this to the 1.1.0 milestone Mar 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vernedeng vernedeng

Labels
type/bug Something is wrong
Projects
None yet
Milestone
1.1.0
Development

Successfully merging a pull request may close this issue.

[INLONG-3332][Sort-Standalone] Fix NP and data race in ClsSink vernedeng/incubator-inlong
2 participants
@healchow @vernedeng