Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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
Collaborator

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
Collaborator

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
Collaborator

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

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

No branches or pull requests

5 participants