Skip to content

Commit

Permalink
Remove "_source_include" and "_source_exclude" from overrides (#4070)
Browse files Browse the repository at this point in the history
  • Loading branch information
codebrain authored and russcam committed Sep 4, 2019
1 parent 4e8a5ea commit 6284623
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,6 @@ public class SearchOverrides : EndpointOverridesBase
"q", //we dont support GET searches
"sort",
"_source",
"_source_include",
"_source_exclude",
"_source_includes",
"_source_excludes",
"track_scores",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ public class UpdateOverrides : EndpointOverridesBase
public override IEnumerable<string> SkipQueryStringParams => new[]
{
"fields",
"_source_include", "_source_exclude",
"_source_includes", "_source_excludes"
};
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,7 @@ public class GlobalOverrides : EndpointOverridesBase
"parent", //can be removed once https://github.com/elastic/elasticsearch/pull/41098 is in
"copy_settings", //this still needs a PR?
"source", // allows the body to be specified as a request param, we do not want to advertise this with a strongly typed method
"timestamp",
"_source_include", "_source_exclude", // can be removed once https://github.com/elastic/elasticsearch/pull/41439 is in
"timestamp"
};
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ public class QueryParameters
{
private static readonly string[] FieldsParams =
{
"fields", "_source_include", "_source_exclude", "_source_includes", "_source_excludes",
"fields", "_source_includes", "_source_excludes",
};

public bool Skip { get; set; }
Expand Down

0 comments on commit 6284623

Please sign in to comment.