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

feat(interactive): Introduce proxy_server into interactive #3778

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

zhanglei1949
Copy link
Collaborator

@zhanglei1949 zhanglei1949 commented May 9, 2024

Originally designed as a standalone graph query system, but in real world scenarios, disaster recovery capabilities is required.

We plan to offering a deployment approach based on master-slave replication, and in order to avoid modifying the core engine, we plan to separate the fault-tolerant features, encapsulating them in a separate server called proxy_server.

The proxy_server is designed to

  • Connect to server interactive workers.
  • Forward the received request to the workers, and gather the responses.
  • Periodically check the healthiness of workers. If a worker are down, proxy will not send request to it anymore. If ALL workers are down, the proxy will return error to client.

It seems that we can not avoid introducing new actor, handler and service definitions.

The usage of proxy_server

./bin/proxy_server -e localhost:10000,localhost:10001

TODO: Currently heartbeat will only return OK, however, we need SNAPSHOT_ID to be returned.

minor

code refactor

refine proxy server

todo: bind to pvc and deploy on internal

try to test on internal

modify dockerfile

able to launch, todo: fix graph creation and loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant