From e4ed9d3a393a1fc2c7a1da1591bd38eff27f5870 Mon Sep 17 00:00:00 2001 From: Sylvain Wallez Date: Mon, 22 Nov 2021 22:46:51 +0100 Subject: [PATCH] Unify Operator and DefaultOperator enums --- output/schema/schema.json | 43 ++++++------------- output/typescript/types.ts | 16 +++---- specification/_global/count/CountRequest.ts | 10 ++--- .../delete_by_query/DeleteByQueryRequest.ts | 4 +- .../_global/explain/ExplainRequest.ts | 5 ++- specification/_global/search/SearchRequest.ts | 4 +- .../update_by_query/UpdateByQueryRequest.ts | 4 +- specification/_types/common.ts | 5 --- .../submit/AsyncSearchSubmitRequest.ts | 4 +- .../IndicesValidateQueryRequest.ts | 10 ++--- 10 files changed, 38 insertions(+), 67 deletions(-) diff --git a/output/schema/schema.json b/output/schema/schema.json index 62acc3f725..0cb242619e 100644 --- a/output/schema/schema.json +++ b/output/schema/schema.json @@ -13717,8 +13717,8 @@ "type": { "kind": "instance_of", "type": { - "name": "DefaultOperator", - "namespace": "_types" + "name": "Operator", + "namespace": "_types.query_dsl" } } }, @@ -14350,8 +14350,8 @@ "type": { "kind": "instance_of", "type": { - "name": "DefaultOperator", - "namespace": "_types" + "name": "Operator", + "namespace": "_types.query_dsl" } } }, @@ -15545,8 +15545,8 @@ "type": { "kind": "instance_of", "type": { - "name": "DefaultOperator", - "namespace": "_types" + "name": "Operator", + "namespace": "_types.query_dsl" } } }, @@ -22065,8 +22065,8 @@ "type": { "kind": "instance_of", "type": { - "name": "DefaultOperator", - "namespace": "_types" + "name": "Operator", + "namespace": "_types.query_dsl" } } }, @@ -29878,8 +29878,8 @@ "type": { "kind": "instance_of", "type": { - "name": "DefaultOperator", - "namespace": "_types" + "name": "Operator", + "namespace": "_types.query_dsl" } } }, @@ -31205,21 +31205,6 @@ } } }, - { - "kind": "enum", - "members": [ - { - "name": "AND" - }, - { - "name": "OR" - } - ], - "name": { - "name": "DefaultOperator", - "namespace": "_types" - } - }, { "generics": [ { @@ -65509,8 +65494,8 @@ "type": { "kind": "instance_of", "type": { - "name": "DefaultOperator", - "namespace": "_types" + "name": "Operator", + "namespace": "_types.query_dsl" } } }, @@ -102422,8 +102407,8 @@ "type": { "kind": "instance_of", "type": { - "name": "DefaultOperator", - "namespace": "_types" + "name": "Operator", + "namespace": "_types.query_dsl" } } }, diff --git a/output/typescript/types.ts b/output/typescript/types.ts index edb362730d..db65f40841 100644 --- a/output/typescript/types.ts +++ b/output/typescript/types.ts @@ -121,7 +121,7 @@ export interface CountRequest extends RequestBase { allow_no_indices?: boolean analyzer?: string analyze_wildcard?: boolean - default_operator?: DefaultOperator + default_operator?: QueryDslOperator df?: string expand_wildcards?: ExpandWildcards ignore_throttled?: boolean @@ -182,7 +182,7 @@ export interface DeleteByQueryRequest extends RequestBase { analyzer?: string analyze_wildcard?: boolean conflicts?: Conflicts - default_operator?: DefaultOperator + default_operator?: QueryDslOperator df?: string expand_wildcards?: ExpandWildcards from?: long @@ -303,7 +303,7 @@ export interface ExplainRequest extends RequestBase { index: IndexName analyzer?: string analyze_wildcard?: boolean - default_operator?: DefaultOperator + default_operator?: QueryDslOperator df?: string lenient?: boolean preference?: string @@ -955,7 +955,7 @@ export interface SearchRequest extends RequestBase { analyze_wildcard?: boolean batched_reduce_size?: long ccs_minimize_roundtrips?: boolean - default_operator?: DefaultOperator + default_operator?: QueryDslOperator df?: string docvalue_fields?: Fields expand_wildcards?: ExpandWildcards @@ -1783,7 +1783,7 @@ export interface UpdateByQueryRequest extends RequestBase { analyzer?: string analyze_wildcard?: boolean conflicts?: Conflicts - default_operator?: DefaultOperator + default_operator?: QueryDslOperator df?: string expand_wildcards?: ExpandWildcards from?: long @@ -1933,8 +1933,6 @@ export type DateMathTime = string export type DateString = string -export type DefaultOperator = 'AND' | 'OR' - export interface DictionaryResponseBase { [key: string]: TValue } @@ -5640,7 +5638,7 @@ export interface AsyncSearchSubmitRequest extends RequestBase { analyze_wildcard?: boolean batched_reduce_size?: long ccs_minimize_roundtrips?: boolean - default_operator?: DefaultOperator + default_operator?: QueryDslOperator df?: string docvalue_fields?: Fields expand_wildcards?: ExpandWildcards @@ -10284,7 +10282,7 @@ export interface IndicesValidateQueryRequest extends RequestBase { all_shards?: boolean analyzer?: string analyze_wildcard?: boolean - default_operator?: DefaultOperator + default_operator?: QueryDslOperator df?: string expand_wildcards?: ExpandWildcards explain?: boolean diff --git a/specification/_global/count/CountRequest.ts b/specification/_global/count/CountRequest.ts index 4301548c1a..0db06802f8 100644 --- a/specification/_global/count/CountRequest.ts +++ b/specification/_global/count/CountRequest.ts @@ -18,14 +18,10 @@ */ import { RequestBase } from '@_types/Base' -import { - DefaultOperator, - ExpandWildcards, - Indices, - Routing -} from '@_types/common' +import { ExpandWildcards, Indices, Routing } from '@_types/common' import { double, long } from '@_types/Numeric' import { QueryContainer } from '@_types/query_dsl/abstractions' +import { Operator } from '@_types/query_dsl/Operator' /** * @rest_spec_name count @@ -40,7 +36,7 @@ export interface Request extends RequestBase { allow_no_indices?: boolean analyzer?: string analyze_wildcard?: boolean - default_operator?: DefaultOperator + default_operator?: Operator df?: string expand_wildcards?: ExpandWildcards ignore_throttled?: boolean diff --git a/specification/_global/delete_by_query/DeleteByQueryRequest.ts b/specification/_global/delete_by_query/DeleteByQueryRequest.ts index 52df29cf6c..59988ab5a2 100644 --- a/specification/_global/delete_by_query/DeleteByQueryRequest.ts +++ b/specification/_global/delete_by_query/DeleteByQueryRequest.ts @@ -20,7 +20,6 @@ import { RequestBase } from '@_types/Base' import { Conflicts, - DefaultOperator, ExpandWildcards, Fields, Indices, @@ -33,6 +32,7 @@ import { QueryContainer } from '@_types/query_dsl/abstractions' import { SlicedScroll } from '@_types/SlicedScroll' import { Time } from '@_types/Time' import { SourceConfigParam } from '@global/search/_types/SourceFilter' +import { Operator } from '@_types/query_dsl/Operator' /** * @rest_spec_name delete_by_query @@ -48,7 +48,7 @@ export interface Request extends RequestBase { analyzer?: string analyze_wildcard?: boolean conflicts?: Conflicts - default_operator?: DefaultOperator + default_operator?: Operator df?: string expand_wildcards?: ExpandWildcards from?: long diff --git a/specification/_global/explain/ExplainRequest.ts b/specification/_global/explain/ExplainRequest.ts index 1cf595141d..46c4870f0f 100644 --- a/specification/_global/explain/ExplainRequest.ts +++ b/specification/_global/explain/ExplainRequest.ts @@ -18,9 +18,10 @@ */ import { RequestBase } from '@_types/Base' -import { DefaultOperator, Fields, Id, IndexName, Routing } from '@_types/common' +import { Fields, Id, IndexName, Routing } from '@_types/common' import { QueryContainer } from '@_types/query_dsl/abstractions' import { SourceConfigParam } from '@global/search/_types/SourceFilter' +import { Operator } from '@_types/query_dsl/Operator' /** * @rest_spec_name explain @@ -35,7 +36,7 @@ export interface Request extends RequestBase { query_parameters: { analyzer?: string analyze_wildcard?: boolean - default_operator?: DefaultOperator + default_operator?: Operator df?: string lenient?: boolean preference?: string diff --git a/specification/_global/search/SearchRequest.ts b/specification/_global/search/SearchRequest.ts index 136e78eb05..b1cee660e7 100644 --- a/specification/_global/search/SearchRequest.ts +++ b/specification/_global/search/SearchRequest.ts @@ -21,7 +21,6 @@ import { Dictionary } from '@spec_utils/Dictionary' import { AggregationContainer } from '@_types/aggregations/AggregationContainer' import { RequestBase } from '@_types/Base' import { - DefaultOperator, ExpandWildcards, Field, Fields, @@ -46,6 +45,7 @@ import { Sort, SortResults } from './_types/sort' import { SourceConfigParam, SourceConfig } from './_types/SourceFilter' import { Suggester } from './_types/suggester' import { TrackHits } from '@global/search/_types/hits' +import { Operator } from '@_types/query_dsl/Operator' /** * @rest_spec_name search @@ -63,7 +63,7 @@ export interface Request extends RequestBase { analyze_wildcard?: boolean batched_reduce_size?: long ccs_minimize_roundtrips?: boolean - default_operator?: DefaultOperator + default_operator?: Operator df?: string docvalue_fields?: Fields expand_wildcards?: ExpandWildcards diff --git a/specification/_global/update_by_query/UpdateByQueryRequest.ts b/specification/_global/update_by_query/UpdateByQueryRequest.ts index 14f7745f5e..4c003237fc 100644 --- a/specification/_global/update_by_query/UpdateByQueryRequest.ts +++ b/specification/_global/update_by_query/UpdateByQueryRequest.ts @@ -20,7 +20,6 @@ import { RequestBase } from '@_types/Base' import { Conflicts, - DefaultOperator, ExpandWildcards, Fields, Indices, @@ -34,6 +33,7 @@ import { Script } from '@_types/Scripting' import { SlicedScroll } from '@_types/SlicedScroll' import { Time } from '@_types/Time' import { SourceConfigParam } from '@global/search/_types/SourceFilter' +import { Operator } from '@_types/query_dsl/Operator' /** * @rest_spec_name update_by_query @@ -49,7 +49,7 @@ export interface Request extends RequestBase { analyzer?: string analyze_wildcard?: boolean conflicts?: Conflicts - default_operator?: DefaultOperator + default_operator?: Operator df?: string expand_wildcards?: ExpandWildcards from?: long diff --git a/specification/_types/common.ts b/specification/_types/common.ts index 1a17dbbcd8..1b5ff62c88 100644 --- a/specification/_types/common.ts +++ b/specification/_types/common.ts @@ -170,11 +170,6 @@ export enum Conflicts { export type Username = string export type Password = string -export enum DefaultOperator { - AND = 0, - OR = 1 -} - export class ElasticsearchResponseBase {} export class ElasticsearchUrlFormatter {} diff --git a/specification/async_search/submit/AsyncSearchSubmitRequest.ts b/specification/async_search/submit/AsyncSearchSubmitRequest.ts index df8c86bdb7..f6fe0a47a4 100644 --- a/specification/async_search/submit/AsyncSearchSubmitRequest.ts +++ b/specification/async_search/submit/AsyncSearchSubmitRequest.ts @@ -21,7 +21,6 @@ import { Dictionary } from '@spec_utils/Dictionary' import { AggregationContainer } from '@_types/aggregations/AggregationContainer' import { RequestBase } from '@_types/Base' import { - DefaultOperator, ExpandWildcards, Field, Fields, @@ -49,6 +48,7 @@ import { } from '@global/search/_types/SourceFilter' import { Suggester } from '@global/search/_types/suggester' import { TrackHits } from '@global/search/_types/hits' +import { Operator } from '@_types/query_dsl/Operator' /** * @rest_spec_name async_search.submit @@ -73,7 +73,7 @@ export interface Request extends RequestBase { analyze_wildcard?: boolean batched_reduce_size?: long ccs_minimize_roundtrips?: boolean - default_operator?: DefaultOperator + default_operator?: Operator df?: string docvalue_fields?: Fields expand_wildcards?: ExpandWildcards diff --git a/specification/indices/validate_query/IndicesValidateQueryRequest.ts b/specification/indices/validate_query/IndicesValidateQueryRequest.ts index a0e6874998..4c83b5a32f 100644 --- a/specification/indices/validate_query/IndicesValidateQueryRequest.ts +++ b/specification/indices/validate_query/IndicesValidateQueryRequest.ts @@ -18,13 +18,9 @@ */ import { RequestBase } from '@_types/Base' -import { - DefaultOperator, - ExpandWildcards, - Indices, - Types -} from '@_types/common' +import { ExpandWildcards, Indices, Types } from '@_types/common' import { QueryContainer } from '@_types/query_dsl/abstractions' +import { Operator } from '@_types/query_dsl/Operator' /** * @rest_spec_name indices.validate_query @@ -41,7 +37,7 @@ export interface Request extends RequestBase { all_shards?: boolean analyzer?: string analyze_wildcard?: boolean - default_operator?: DefaultOperator + default_operator?: Operator df?: string expand_wildcards?: ExpandWildcards explain?: boolean