Skip to content
This repository has been archived by the owner on Jan 7, 2023. It is now read-only.

[analyzer] update get_execute_time() #202

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Tanyy
Copy link
Contributor

@Tanyy Tanyy commented Mar 15, 2017

we cannot get execute time if head node not a worker.
so, we shall use one of the ceph mon node to get execute time .

we cannot get execute time if head node not a worker.
so, we shall use one of the ceph mon node to get execute time .
@xuechendi
Copy link
Contributor

nice catch, how about we use the first client? Since cetune is also able to test on non-ceph cluster, so I think client is a must-exist item when extract execute time. What do you think?

@Tanyy
Copy link
Contributor Author

Tanyy commented Mar 15, 2017 via email

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