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] Optimize client http request #4381

Closed
2 tasks done
leosanqing opened this issue May 25, 2022 · 2 comments · Fixed by #4461
Closed
2 tasks done

[Improve][Manager] Optimize client http request #4381

leosanqing opened this issue May 25, 2022 · 2 comments · Fixed by #4461
Assignees
@leosanqing @healchow
Labels
component/manager type/improve
Milestone
1.2.0

Comments

@leosanqing
Copy link
Contributor

Description

  1. Optimize client http request，Simplified method
  2. use Jackson instead of gson
  3. add some junit tests

InLong Component

InLong Manager

Are you willing to submit PR?

  • Yes, I am willing to submit a PR!

Code of Conduct
@dockerzhang
Copy link
Contributor

@leosanqing do you still work on this issue?

@leosanqing
Copy link
Contributor Author

@leosanqing do you still work on this issue?

Yes, I have completed the development, but when merging the code, I found that some util classes disappeared, waiting for healchow to complete the new util class

@healchow

@leosanqing leosanqing mentioned this issue May 31, 2022
Merged
1 task
@healchow healchow added this to the 1.2.0 milestone May 31, 2022
@healchow healchow self-assigned this Jun 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@leosanqing leosanqing

@healchow healchow

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

Successfully merging a pull request may close this issue.

[INLONG-4381][Manager] Optimize the HTTP request in manager client leosanqing/incubator-inlong
3 participants
@dockerzhang @leosanqing @healchow