-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[exporter/logicmonitor] Update the lm-data-sdk-go version to v1.2.0 and fix error handling issue #30431
[exporter/logicmonitor] Update the lm-data-sdk-go version to v1.2.0 and fix error handling issue #30431
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this need a changelog?
I don't think so. |
cc: @khyatigandhi6 |
Changes looks good to me. No change log required as error handling is done as internal implementation. |
if ingestResponse.Success { | ||
return nil | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't understand why this has been removed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because, with the updated SDK implementation if there is non empty error, then this success
flag in the response is always false, so no need to check that explicitly and if error is nil
then it is always true.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the explanation :) LGTM!
…ctor-contrib into fix-error-handling
…nd fix error handling issue (open-telemetry#30431) **Description:** <Describe what has changed.> Fixing a bug - Using the updated version (v1.2.0) of `lm-data-sdk-go` causes each error to be considered as a permanent. This PR updates the version as well as changes the error handling to fix this issue. **Link to tracking Issue:** NA **Testing:** Unit and functional testing. **Documentation:** NA
Description:
Fixing a bug - Using the updated version (v1.2.0) of
lm-data-sdk-go
causes each error to be considered as a permanent. This PR updates the version as well as changes the error handling to fix this issue.Link to tracking Issue: NA
Testing: Unit and functional testing.
Documentation: NA