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
[Connector API] Support filtering connectors by service type and a query #105178
[Connector API] Support filtering connectors by service type and a query #105178
Conversation
Hi @jedrazb, I've created a changelog YAML for you. |
@elasticsearchmachine run elasticsearch-ci/part-3 |
Pinging @elastic/ent-search-eng (Team:Enterprise Search) |
d2016c6
to
59248e1
Compare
@elasticsearchmachine run elasticsearch-ci/bwc-snapshots |
@elasticsearchmachine run elasticsearch-ci/8.13.0 / bwc-snapshots |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just one comment
"type": "list", | ||
"description": "A comma-separated list of connector name(s) to fetch connector documents for" | ||
}, | ||
"service_type": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why don't we support this as a list?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
uhh that's a good question, I can change this into a list actually. I wasn't sure if we need to support filtering by multiple jobtypes?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm adding support for multiple service types and merging :) it's a small change
@elasticsearchmachine run elasticsearch-ci/part-3 |
Changes
service_type
in list endpointname
,index_name
anddescription
field