Skip to content

More debugging info for significant_text (backport of #72727)#72895

Merged
nik9000 merged 2 commits intoelastic:7.xfrom
nik9000:sig_text_extra_7_x
May 11, 2021
Merged

More debugging info for significant_text (backport of #72727)#72895
nik9000 merged 2 commits intoelastic:7.xfrom
nik9000:sig_text_extra_7_x

Conversation

@nik9000
Copy link
Copy Markdown
Member

@nik9000 nik9000 commented May 10, 2021

Adds some extra debugging information to make it clear that you are
running significant_text. Also adds some using timing information
around the _source fetch and the terms accumulation. This lets you
calculate a third useful timing number: the analysis time. It is
collect_ns - fetch_ns - accumulation_ns.

This also adds a half dozen extra REST tests to get a fairly
comprehensive set of the operations this supports. It doesn't cover all
of the significance heuristic parsing, but its certainly much better
than what we had.

Adds some extra debugging information to make it clear that you are
running `significant_text`. Also adds some using timing information
around the `_source` fetch and the `terms` accumulation. This lets you
calculate a third useful timing number: the analysis time. It is
`collect_ns - fetch_ns - accumulation_ns`.

This also adds a half dozen extra REST tests to get a *fairly*
comprehensive set of the operations this supports. It doesn't cover all
of the significance heuristic parsing, but its certainly much better
than what we had.
@nik9000 nik9000 merged commit 70e7946 into elastic:7.x May 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant