Skip to content

Commit

Permalink
Fix async_search query parameter validation (#3167)
Browse files Browse the repository at this point in the history
  • Loading branch information
pquentin authored Nov 26, 2024
1 parent 2942c94 commit db7943e
Show file tree
Hide file tree
Showing 8 changed files with 67 additions and 68 deletions.
25 changes: 10 additions & 15 deletions output/openapi/elasticsearch-openapi.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

25 changes: 10 additions & 15 deletions output/openapi/elasticsearch-serverless-openapi.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

31 changes: 17 additions & 14 deletions output/schema/schema-serverless.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

31 changes: 17 additions & 14 deletions output/schema/schema.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

7 changes: 0 additions & 7 deletions output/schema/validation-errors.json
Original file line number Diff line number Diff line change
Expand Up @@ -49,15 +49,8 @@
"type_alias definition _global.search._types:Suggest - Expected 1 generic parameters but got 0"
]
},
"async_search.status": {
"request": [
"Request: missing json spec query parameter 'keep_alive'"
],
"response": []
},
"async_search.submit": {
"request": [
"Request: query parameter 'scroll' does not exist in the json spec",
"interface definition _types:QueryVectorBuilder - Property text_embedding is a single-variant and must be required",
"type_alias definition _spec_utils:PipeSeparatedFlags / union_of / instance_of - No type definition for '_spec_utils.PipeSeparatedFlags:T'"
],
Expand Down
2 changes: 1 addition & 1 deletion output/typescript/types.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

9 changes: 9 additions & 0 deletions specification/async_search/status/AsyncSearchStatusRequest.ts
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@

import { RequestBase } from '@_types/Base'
import { Id } from '@_types/common'
import { Duration } from '@_types/Time'

/**
* Get the async search status.
Expand All @@ -36,4 +37,12 @@ export interface Request extends RequestBase {
/** A unique identifier for the async search. */
id: Id
}
query_parameters: {
/**
* Specifies how long the async search needs to be available.
* Ongoing async searches and any saved search results are deleted after this period.
* @server_default 5d
*/
keep_alive?: Duration
}
}
5 changes: 3 additions & 2 deletions specification/async_search/submit/AsyncSearchSubmitRequest.ts
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,9 @@ import { Duration } from '@_types/Time'
* @doc_id async-search
* @doc_tag search
*/
// NOTE: this is a SearchRequest with 3 added parameters: wait_for_completion_timeout, keep_on_completion and keep_alive
// NOTE: this is a SearchRequest with:
// * 3 added parameters: wait_for_completion_timeout, keep_on_completion and keep_alive
// * 1 removed parameters: scroll
export interface Request extends RequestBase {
path_parts: {
index?: Indices
Expand Down Expand Up @@ -122,7 +124,6 @@ export interface Request extends RequestBase {
/** @server_default true */
request_cache?: boolean
routing?: Routing
scroll?: Duration
search_type?: SearchType
stats?: string[]
stored_fields?: Fields
Expand Down

0 comments on commit db7943e

Please sign in to comment.