gemini-code-assist[bot] commented on code in PR #38381:
URL: https://github.com/apache/beam/pull/38381#discussion_r3196865454
##########
sdks/go/pkg/beam/io/fhirio/common.go:
##########
@@ -130,9 +130,9 @@ func (c *fhirStoreClientImpl) search(storePath,
resourceType string, queries map
// Pass nil as the body because search parameters are passed via
queryParams,
// and the new API expects an io.Reader instead of a specific struct.
if resourceType == "" {
- return c.fhirService().Search(storePath, nil).Do(queryParams...)
+ return c.fhirService().Search(storePath,
strings.NewReader("")).Do(queryParams...)
Review Comment:
The comment on line 130 is now inaccurate as it still mentions passing `nil`
as the body. Since the code has been updated to pass an empty reader to satisfy
the new API requirements, the comment should be updated to reflect this change
(e.g., "Pass an empty reader as the body...").
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
