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

HG: add timeout test #393

Open
soumagne opened this issue Sep 7, 2020 · 2 comments
Open

HG: add timeout test #393

soumagne opened this issue Sep 7, 2020 · 2 comments

Comments

@soumagne
Copy link
Member

soumagne commented Sep 7, 2020

Not really high-priority but once in a while we have timeout issues (not at the moment) and it would be fairly easy to measure how much time is spent in routines that take a timeout.

@lokesharo
Copy link

I am using Mercury for RPC in my project.
Link states that timeouts are not supported by mercury due to metadata overhead. Hence, our project uses our own timeouts and then invoke HG_Cancel() in case timer expires.

What timeout tests are we exactly talking about in the current task? Has the timeouts been implemented in the latest release or is in pipeline?

@soumagne
Copy link
Member Author

This task was meant to add tests that check existing timeouts that are passed to functions such as HG_Progress() / HG_Trigger(). We do not plan to support passing of timeouts to HG_Forward() / HG_Respond() at this time as this would require tracking of operations internally. If you need something that facilitates implementation of timeouts or need timers, you can look at using mercury_time.h though. Using your own timeout and calling cancel is what we intend you to do.

@soumagne soumagne modified the milestones: mercury-2.1.0, future Oct 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants