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

[Improve][Manager] Optimizing source module code structure #2669

Closed
2 tasks done
herywang opened this issue Feb 23, 2022 · 0 comments · Fixed by #2671
Closed
2 tasks done

[Improve][Manager] Optimizing source module code structure #2669

herywang opened this issue Feb 23, 2022 · 0 comments · Fixed by #2671
Assignees
@herywang
Labels
component/manager type/improve
Milestone
1.1.0

Comments

@herywang
Copy link
Contributor

Description

Description

Optimizing inlong-manager stream source module code structure.

We will add other stream source support in the near future, so I optimized the public method and extract them into an abstract class named AbstractStreamSourceOperation

InLong Component

InLong Manager

Are you willing to submit PR?

  • Yes, I am willing to submit a PR!

Code of Conduct
@herywang herywang mentioned this issue Feb 23, 2022
Merged
healchow pushed a commit that referenced this issue Feb 24, 2022
@herywang
[INLONG-2669][Manager] Optimizing inlong-manager source module code s…
9d0479c 
…tructure (#2669)
@healchow healchow changed the title [Improve][Manager] Optimizing inlong-manager source module code structure. [Improve][Manager] Optimizing source module code structure Mar 12, 2022
@healchow healchow added this to the 1.1.0 milestone Mar 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@herywang herywang

Labels
component/manager type/improve
Projects
None yet
Milestone
1.1.0
Development

Successfully merging a pull request may close this issue.

[INLONG-2669][Manager] Optimizing inlong-manager source module code s…
2 participants
@herywang @healchow