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
Add response close go client #2828
Add response close go client #2828
Conversation
Thank you, @sheltie-fusafusa, for contributing! @VachaShah could you please review for technical accuracy? Thank you! |
The "Go package documentation" calls the Close method. https://pkg.go.dev/github.com/opensearch-project/opensearch-go#NewClient Signed-off-by: sheltie <sheltie.fusafusa@gmail.com>
Signed-off-by: sheltie <sheltie.fusafusa@gmail.com>
This reverts commit 6852b1c. Signed-off-by: sheltie <sheltie.fusafusa@gmail.com>
115c245
to
11074f6
Compare
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.
LGTM, thank you @sheltie-fusafusa!
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.
LGTM. Thank you!
* I think response need to close. The "Go package documentation" calls the Close method. https://pkg.go.dev/github.com/opensearch-project/opensearch-go#NewClient Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * delete defer Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * Revert "delete defer" This reverts commit 6852b1c. Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> --------- Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> (cherry picked from commit 94e11ab)
* I think response need to close. The "Go package documentation" calls the Close method. https://pkg.go.dev/github.com/opensearch-project/opensearch-go#NewClient Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * delete defer Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * Revert "delete defer" This reverts commit 6852b1c. Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> --------- Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> (cherry picked from commit 94e11ab)
* I think response need to close. The "Go package documentation" calls the Close method. https://pkg.go.dev/github.com/opensearch-project/opensearch-go#NewClient Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * delete defer Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * Revert "delete defer" This reverts commit 6852b1c. Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> --------- Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> (cherry picked from commit 94e11ab)
* I think response need to close. The "Go package documentation" calls the Close method. https://pkg.go.dev/github.com/opensearch-project/opensearch-go#NewClient Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * delete defer Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * Revert "delete defer" This reverts commit 6852b1c. Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> --------- Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> (cherry picked from commit 94e11ab)
* I think response need to close. The "Go package documentation" calls the Close method. https://pkg.go.dev/github.com/opensearch-project/opensearch-go#NewClient Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * delete defer Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * Revert "delete defer" This reverts commit 6852b1c. Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> --------- Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> (cherry picked from commit 94e11ab)
* I think response need to close. The "Go package documentation" calls the Close method. https://pkg.go.dev/github.com/opensearch-project/opensearch-go#NewClient Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * delete defer Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * Revert "delete defer" This reverts commit 6852b1c. Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> --------- Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> (cherry picked from commit 94e11ab)
* I think response need to close. The "Go package documentation" calls the Close method. https://pkg.go.dev/github.com/opensearch-project/opensearch-go#NewClient Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * delete defer Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * Revert "delete defer" This reverts commit 6852b1c. Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> --------- Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> (cherry picked from commit 94e11ab)
* I think response need to close. The "Go package documentation" calls the Close method. https://pkg.go.dev/github.com/opensearch-project/opensearch-go#NewClient Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * delete defer Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * Revert "delete defer" This reverts commit 6852b1c. Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> --------- Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> (cherry picked from commit 94e11ab) Co-authored-by: sheltie <sheltie.fusafusa@gmail.com>
* I think response need to close. The "Go package documentation" calls the Close method. https://pkg.go.dev/github.com/opensearch-project/opensearch-go#NewClient Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * delete defer Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * Revert "delete defer" This reverts commit 6852b1c. Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> --------- Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> (cherry picked from commit 94e11ab) Co-authored-by: sheltie <sheltie.fusafusa@gmail.com>
* I think response need to close. The "Go package documentation" calls the Close method. https://pkg.go.dev/github.com/opensearch-project/opensearch-go#NewClient Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * delete defer Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * Revert "delete defer" This reverts commit 6852b1c. Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> --------- Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> (cherry picked from commit 94e11ab) Co-authored-by: sheltie <sheltie.fusafusa@gmail.com>
* I think response need to close. The "Go package documentation" calls the Close method. https://pkg.go.dev/github.com/opensearch-project/opensearch-go#NewClient Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * delete defer Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * Revert "delete defer" This reverts commit 6852b1c. Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> --------- Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> (cherry picked from commit 94e11ab) Co-authored-by: sheltie <sheltie.fusafusa@gmail.com>
* I think response need to close. The "Go package documentation" calls the Close method. https://pkg.go.dev/github.com/opensearch-project/opensearch-go#NewClient Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * delete defer Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * Revert "delete defer" This reverts commit 6852b1c. Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> --------- Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> (cherry picked from commit 94e11ab) Co-authored-by: sheltie <sheltie.fusafusa@gmail.com>
* I think response need to close. The "Go package documentation" calls the Close method. https://pkg.go.dev/github.com/opensearch-project/opensearch-go#NewClient Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * delete defer Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * Revert "delete defer" This reverts commit 6852b1c. Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> --------- Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> (cherry picked from commit 94e11ab) Co-authored-by: sheltie <sheltie.fusafusa@gmail.com>
* I think response need to close. The "Go package documentation" calls the Close method. https://pkg.go.dev/github.com/opensearch-project/opensearch-go#NewClient Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * delete defer Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> * Revert "delete defer" This reverts commit 6852b1c. Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> --------- Signed-off-by: sheltie <sheltie.fusafusa@gmail.com> (cherry picked from commit 94e11ab) Co-authored-by: sheltie <sheltie.fusafusa@gmail.com>
Signed-off-by: Takumi Hamada sheltie.fusafusa@gmail.com
Description
I think response need to close.
because "Go package documentation" calls the Close method.
https://pkg.go.dev/github.com/opensearch-project/opensearch-go#NewClient
Issues Resolved
Checklist
For more information on following Developer Certificate of Origin and signing off your commits, please check here.