-
Notifications
You must be signed in to change notification settings - Fork 7
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
Cleaning up extra printed messages added during debugging #14
Cleaning up extra printed messages added during debugging #14
Conversation
rpc/src/main_rpc_server.cpp
Outdated
/* Build response attributes - the same UUID should be used to send the response | ||
* it is also possible to send the response outside of the callback context */ | ||
auto builder = UAttributesBuilder::response( | ||
message.attributes().source(), |
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.
source is sink and sink is source so flip these two
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.
fixed
rpc/src/main_rpc_server.cpp
Outdated
auto builder = UAttributesBuilder::response( | ||
message.attributes().source(), | ||
message.attributes().sink(), | ||
UPriority::UPRIORITY_CS0, |
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.
This needs to be message.attributes.priority().
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.
fixed
4901523
to
244edfe
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.
There is a bug for priority
rpc/src/main_rpc_server.cpp
Outdated
auto builder = UAttributesBuilder::response( | ||
message.attributes().sink(), | ||
message.attributes().source(), | ||
message.attributes.priority(), |
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.
message.attributes.priority(), | |
message.attributes().priority(), |
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.
Fixed
While debugging the RPC timeout issue, several extra messages were added and the reply data format were changed. Changing these back so the example is cleaner and outputs the expected time value.
244edfe
to
890ad14
Compare
While debugging the RPC timeout issue, several extra messages were added and the reply data format were changed. Changing these back so the example is cleaner and outputs the expected time value.