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

Requests are now timed #9583

Closed
wants to merge 2 commits into from
Closed

Conversation

alexksikes
Copy link
Contributor

When asking for term statistics, generating term vectors on the fly or with
dfs set to true, some requests may take a while, so it is useful to know
exactly how long.

When asking for term statistics, generating term vectors on the fly or with
`dfs` set to `true`, some requests may take a while, so it is useful to know
exactly how long.
@jpountz
Copy link
Contributor

jpountz commented Feb 10, 2015

Why do we need to serialize the start time, can it happen that the start and end times might be computed on different machines? If yes it is a concern to me since it makes the time measurement quite unreliable.

@jpountz
Copy link
Contributor

jpountz commented Feb 16, 2015

LGTM

alexksikes added a commit that referenced this pull request Feb 16, 2015
When asking for term statistics, generating term vectors on the fly or with
`dfs` set to `true`, some requests may take a while, so it is useful to know
exactly how long.

Closes #9583
@alexksikes alexksikes deleted the feature/tvs-timed branch February 16, 2015 14:02
@clintongormley clintongormley changed the title Term Vectors: requests are now timed Requests are now timed Jun 7, 2015
@clintongormley clintongormley added :Search/Search Search-related issues that do not fall into other categories and removed :Term Vectors labels Feb 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>enhancement :Search/Search Search-related issues that do not fall into other categories v1.5.0 v2.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants