-
Notifications
You must be signed in to change notification settings - Fork 321
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
why named rpc ? #54
Comments
RPC is remote procedure call. This is exactly what it does in unit. |
This so-called rpc is just for callback, and a msg type(_NXT_PORT_MSG_RPC_READY, _NXT_PORT_MSG_RPC_ERROR). Rpc is too big a name for this. Or maybe rpc_callback could be used. |
RPC is a form of IPC and "remote" doesn't necessarily mean physically remote, but in a different virtual address space. |
4141done
pushed a commit
to 4141done/unit
that referenced
this issue
Sep 21, 2023
This closes PR nginx#54 on GitHub.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
As for now, rpc is just used callback across. It's confused me. What about rename it to callback?
And port is confusing too. Maybe ipc should be used ? Just wondering.
The text was updated successfully, but these errors were encountered: