Skip to content

Conversation

@martijnvg
Copy link
Member

PR for #24958

@martijnvg martijnvg added :Search Relevance/Percolator Reverse search: find queries that match a document review >test Issues or PRs that are addressing/adding tests v6.0.0 labels May 30, 2017
Copy link
Contributor

@s1monw s1monw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM left 2 comments

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we use doc as the type?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am curious, why is it less now?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That is because with a single type, the _type indexed and stored fields are left out.

@martijnvg martijnvg force-pushed the percolator_stop_using_single_type_setting branch 2 times, most recently from e3680f1 to a4782f5 Compare May 31, 2017 07:10
@martijnvg martijnvg force-pushed the percolator_stop_using_single_type_setting branch from a4782f5 to 6945d7b Compare May 31, 2017 07:11
@martijnvg martijnvg merged commit 6945d7b into elastic:master May 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Search Relevance/Percolator Reverse search: find queries that match a document >test Issues or PRs that are addressing/adding tests v6.0.0-alpha2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants