Skip to content
This repository was archived by the owner on May 31, 2025. It is now read-only.

xmlrpc_manager: use SteadyTime for timeout#1134

Merged
dirk-thomas merged 1 commit intoros:lunar-develfrom
flixr:xmlrpc_steady_timeout
Aug 14, 2017
Merged

xmlrpc_manager: use SteadyTime for timeout#1134
dirk-thomas merged 1 commit intoros:lunar-develfrom
flixr:xmlrpc_steady_timeout

Conversation

@flixr
Copy link
Copy Markdown
Contributor

@flixr flixr commented Aug 12, 2017

Use the SteadyTime instead of WallTime when checking if a cached client is dead.

@dirk-thomas
Copy link
Copy Markdown
Member

How will this affect the behavior when e.g. using sim time? I could imagine that existing connections would be ended when simulation is paused.

@flixr
Copy link
Copy Markdown
Contributor Author

flixr commented Aug 14, 2017

How will this affect the behavior when e.g. using sim time?

Not at all as long as the system time doesn't change.

I could imagine that existing connections would be ended when simulation is paused.

I'm not exactly sure when a new xmlrpc client is needed, but since it was using WallTime before, it would have had the same effect.

So the only difference is that you don't get a timeout if you set the system time forward.

@dirk-thomas
Copy link
Copy Markdown
Member

Nvm. Since it was using wall time before I wasn't affected by sim time anyway.

Thanks.

@dirk-thomas dirk-thomas merged commit 940945c into ros:lunar-devel Aug 14, 2017
@flixr flixr deleted the xmlrpc_steady_timeout branch November 24, 2017 17:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants