raled573343 opened a new pull request, #362:
URL: https://github.com/apache/hugegraph-computer/pull/362
## Purpose of the PR
- close #361
<!--
Please explain more context in this section, clarify why the changes are
needed.
For example:
- If you propose a new API, clarify the use case for a new API.
- If you fix a bug, you can clarify why it is a bug, and should associated
with an issue.
-->
This PR fixes the issue where schema requests fail due to missing `http://`
prefix in the server address. The PD service returns addresses as `host:port`
without a scheme, and the HTTP client in Go requires an absolute URL. This
change adds the missing prefix in both `testServerIsValid` and
`GetHugegraphSchema` functions.
## Main Changes
- In `vermeer/apps/common/hugegraph_tools.go`:
- Updated `testServerIsValid`: URL construction now uses
`"http://%v/graphspaces/..."`.
- Updated `GetHugegraphSchema`: URL construction now uses
`"http://%v/graphspaces/..."`.
## Verifying these changes
- [x] Trivial rework / code cleanup without any test coverage. (No Need)
- [ ] Already covered by existing tests, such as *(please modify tests
here)*.
- [ ] Need tests and can be verified as follows.
<!-- Please provide more details about verification
For example:
- If you test manually, please provide related screenshot.
-->
Manual verification was performed by running the service with the fix; the
schema endpoint now responds correctly with HTTP 200.
## Does this PR potentially affect the following parts?
- [x] Nope
- [ ] Dependencies (add/update license info)
- [ ] Modify configurations
- [ ] The public API
- [ ] Other affects (typed here)
## Documentation Status
- [x] `Doc - No Need` <!-- Your PR changes don't impact/need docs -->
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
