[App Search] Show process crawls together with crawl requests #111398
Conversation
orhantoy
added
Feature:Plugins
release_note:skip
| onReceiveCrawlerData: (_, { data: { events } }) => events, | ||
| }, | ||
| ], | ||
| mostRecentCrawlRequest: [ |
There was a problem hiding this comment.
It seems weird to me that we get the mostRecentCrawl request from the API rather than grabbing it from the list of all events ourselves. Is there a circumstance in which the value returned by the API for mostRecentCrawlRequest would not be the first non-skipped crawl request?
There was a problem hiding this comment.
The new API in use here returns events, which is a combined list of process crawls and crawl requests. So the list of events could potentially only consist of process crawls, meaning we can no longer compute the most recent crawl request.
|
@elasticmachine merge upstream |
💚 Build Succeeded
Metrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: |
orhantoy
deleted the
list-process-crawls-together-with-crawl-requests
branch
💚 Backport successful
This backport PR will be merged automatically after passing CI. |
Co-authored-by: Orhan Toy <[email protected]>
Summary
We want to show the user that a process crawl is being run in the background and the status of it. We already do that for crawl requests, which represent regular crawls.