-
Notifications
You must be signed in to change notification settings - Fork 78
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
Fix rpc_benchmark and add fcall mode #6173
Fix rpc_benchmark and add fcall mode #6173
Conversation
b7a7f74
to
561d14a
Compare
src/rpc/rpc_benchmark.js
Outdated
@@ -110,14 +117,15 @@ schema.compile(); | |||
|
|||
// create rpc | |||
const rpc = new RPC({ | |||
schema: schema, | |||
client_factory: api.client_factory_from_schema(schema), |
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.
Hmm, I want to make sure rpc
module is usable even without api
module.
Do you think this can be done?
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.
Ok we have it now in a sepetate PR: #6178
We can wait to merge this one untill the other one is merged
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.
Rebased against master
and removed dependency with the api
module
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.
See one comment, but overall this is good. Any results from running it?
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.
Please rebase master
61a933a
to
9f9d9c6
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.
Looks great. Do you have the results of running this between the new fcall that clones messages and the previous which was using JSON.stringify/parse?
No, as this change was merged to master after the fcall changes |
Also move the start promise chain to async/await Signed-off-by: nb-ohad <mitrani.ohad@gmail.com>
9f9d9c6
to
767e785
Compare
Explain the changes
--fcall
flag to test RPC overfcalll
connection (process acts as both server and client)Issues: Fixed #xxx / Gap #xxx
Testing Instructions: