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][SDK] Improve response attr parsing in Golang SDK #9174

Closed
2 tasks done
gunli opened this issue Oct 31, 2023 · 0 comments · Fixed by #9175
Closed
2 tasks done

[Improve][SDK] Improve response attr parsing in Golang SDK #9174

gunli opened this issue Oct 31, 2023 · 0 comments · Fixed by #9175
Assignees
@gunli
Labels
type/improve
Milestone
1.10.0

Comments

@gunli
Copy link
Contributor

gunli commented Oct 31, 2023

Description

Currently, when we parse the attr in response, we use Strings.Spilt() to parse, which will alloc new strings in runtime, and this is a frequently called method, and will finally decrease the perf of the SDK. We need to improve it.

InLong Component

InLong SDK

Are you willing to submit PR?

  • Yes, I am willing to submit a PR!

Code of Conduct
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gunli gunli

Labels
type/improve
Projects
None yet
Milestone
1.10.0
Development

Successfully merging a pull request may close this issue.

[INLONG-9174][SDK] Optimize response attr parsing in Golang SDK gunli/inlong
2 participants
@gunli @dockerzhang