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.

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

bad queue length strconv.ParseInt call #6178

Closed
jonsmeep opened this issue Sep 19, 2024 · 1 comment · Fixed by #6235
Closed

bad queue length strconv.ParseInt call #6178

jonsmeep opened this issue Sep 19, 2024 · 1 comment · Fixed by #6235

Comments

@jonsmeep
Copy link

Current code reads:
metricValue, err := strconv.ParseInt(output.Attributes[string(awsSqsQueueMetric)], 10, 32)

that 32 ought to be a 64.

metricValue, err := strconv.ParseInt(output.Attributes[string(awsSqsQueueMetric)], 10, 32)

@JorTurFer
Copy link
Member

Hello
Which is the issue with using int32 here? Are you using a number over int32 max value?

@github-project-automation github-project-automation bot moved this from To Triage to Ready To Ship in Roadmap - KEDA Core Nov 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Ready To Ship
Development

Successfully merging a pull request may close this issue.

2 participants