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

Adds support for wildcards in selected fields #7061

Closed

Conversation

alexksikes
Copy link
Contributor

This could useful to generate all term vectors or a chosen set of them.

This commit simply adds the option to retrieve or generate all possible term
vectors of a document, without having to specify a set of selected fields.
This could be useful for MLT with items, for example, to provide a nice
default if users do not specify any field.
@jpountz
Copy link
Contributor

jpountz commented Jul 30, 2014

The change looks good but I think it would be more consistent with other APIs to accept wildcards in field names than having a generate_all flag? (See eg. http://www.elasticsearch.org/guide/en/elasticsearch/reference/current/query-dsl-multi-match-query.html#_literal_fields_literal_and_per_field_boosting)

@jpountz jpountz removed the review label Jul 30, 2014
@@ -339,7 +339,7 @@ public void writeTo(StreamOutput out) throws IOException {
public static enum Flag {
// Do not change the order of these flags we use
// the ordinal for encoding! Only append to the end!
Positions, Offsets, Payloads, FieldStatistics, TermStatistics;
Positions, Offsets, Payloads, FieldStatistics, TermStatistics, GenerateAll;
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't be needed anymore?

@jpountz
Copy link
Contributor

jpountz commented Jul 30, 2014

@alexksikes left some comments

@jpountz
Copy link
Contributor

jpountz commented Jul 30, 2014

LGTM

@jpountz jpountz removed the review label Jul 30, 2014
alexksikes added a commit that referenced this pull request Jul 30, 2014
This could useful to generate all term vectors or a chosen set of them.

Closes #7061
alexksikes added a commit that referenced this pull request Sep 8, 2014
This could useful to generate all term vectors or a chosen set of them.

Closes #7061
@clintongormley clintongormley changed the title Term Vectors API: generate all possible TVs of a document Term Vectors: Fenerate all possible TVs of a document Sep 8, 2014
@alexksikes alexksikes changed the title Term Vectors: Fenerate all possible TVs of a document Term Vectors: Generate all possible TVs of a document Sep 16, 2014
@alexksikes alexksikes deleted the feature/termvectors-generate-all branch October 1, 2014 09:01
@alexksikes alexksikes changed the title Term Vectors: Generate all possible TVs of a document Term Vectors API: adds support for wildcards in selected fields Oct 1, 2014
@clintongormley clintongormley changed the title Term Vectors API: adds support for wildcards in selected fields Adds support for wildcards in selected fields 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.4.0.Beta1 v2.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants