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

ES-compatible API #2653

Closed
26 of 28 tasks
guilload opened this issue Jan 13, 2023 · 9 comments
Closed
26 of 28 tasks

ES-compatible API #2653

guilload opened this issue Jan 13, 2023 · 9 comments
Assignees

Comments

@guilload
Copy link
Member

guilload commented Jan 13, 2023

Endpoints:

Search parameters:

Query DSL:

Aggregation DSL:

Other missing elastic features that we might discuss:

@guilload
Copy link
Member Author

guilload commented Jun 13, 2023

Bumping priority for:

  • Query string with analyze_wildcard => under discussion
  • Multi-match query

@esatterwhite
Copy link
Collaborator

esatterwhite commented Jun 21, 2023

One thing I'm noticing missing from the _search api is the ability to search across multiple indicies

// curl -XPOST http://localhost:7200/api/v1/_elastic/ac*/_search -d '{"query": { "match_all": {} }, "size": 1}'
{
  "status": 404,
  "error": {
    "caused_by": null,
    "reason": "Index `ac*` does not exist.",
    "stack_trace": null,
    "type": null
  }
}
// curl -XPOST http://localhost:7200/api/v1/_elastic/foo,bar/_search -d '{"query": { "match_all": {} }, "size": 1}'
{
  "message": "Route not found"
}

@fmassot
Copy link
Contributor

fmassot commented Jun 21, 2023

Thanks for the feedback. Just confirming that search across multiple indices is not yet supported.

@esatterwhite
Copy link
Collaborator

The Validation for index names is a little different than what elasticsearch allows.

^[a-zA-Z][a-zA-Z0-9-_]{2,254}$ doesn't allow for a dot (period) . Which is allowed and common in elastic index names.

@esatterwhite
Copy link
Collaborator

It looks like the _bulk endpoint is lacking the ability to create missing indexes as well

@fulmicoton fulmicoton changed the title ES-compatible search API ES-compatible API Aug 15, 2023
@esatterwhite
Copy link
Collaborator

It looks like the / endpoint will need to support the http method HEAD. As far as I can tell, most elasticsearch client do this when trying to establish a connection to verify the server is responsive.

@fulmicoton
Copy link
Contributor

Commenting about the HEAD request. @fmassot fixed it.

@esatterwhite
Copy link
Collaborator

I think we can probably call this done for our purposes @fulmicoton, what do you thin?

@fulmicoton
Copy link
Contributor

If you have everything you need I think we can close it yes!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants