-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Dataset quality] Replaced Estimated Data section by doc count in serverless deployments #180582
Comments
Pinging @elastic/obs-ux-logs-team (Team:obs-ux-logs) |
Hey @mdbirnstiehl we are changing the ![]() What do you think about changing
? |
I think those are fine, with a period at the end of the tool tip. |
@yngrdyn is this not blocked by https://elasticco.atlassian.net/browse/ES-7700? |
Partially, I think story is still open because the storage size was removed. The api has been developed including only the doc count https://github.com/elastic/elasticsearch-serverless/pull/1482. The idea of this story is to replace the |
some quick points here, I would suggest we wait on this until #182873 is merged.
I would suggest we then make this one more thing that will be affected is the |
closes #182873 ## 📝 Summary After implementing #179172 its now time to make the `Estimated Data` panel reactive to all table filters as well. This PR is the last step into making the summary panel reactive to all table filters. As part of the implementation the following has been done: 1. The summary panel state machine is completely removed as its no longer needed and we now rely on the dataset quality state machine. 2. The Estimated Size panel is now reactive to all filters and show a sum of the dataset sizes displayed in the table. 3. The `Size` column now shows the size for each dataset within the given time range. 4. The `Size` column and the `Estimated data` panel are now both hidden on `serverless` as they both rely on the `stats` API which is not yet available on serverless. 5. As part of this ticket #180582 we should start showing the number of docs instead of the size on serverless. ## 🎥 Demo https://github.com/elastic/kibana/assets/11225826/a4a4e013-e514-480c-aa4e-eb9d6ff10b82 --------- Co-authored-by: kibanamachine <[email protected]>
@yngrdyn @gbamparop this can be closed now right? |
I think so as the estimated size is now displayed in Serverless, @flash1293 ? |
In #178954 and #179479 we hid the
Estimated data
section of the summary because_stats
api was not available in serverless - #178955 tracks adding back actual doc size, this will most likely take longer to be resolved.The ES team is working on enabling a new API to get the count of documents per index in serverless and the work is captured in here.
Once available, this should be used on serverless instead of actual index size to give an indication of data volume.
✔️ Acceptance criteria
Estimated data
section is shown for serverless deployments with counts of documents instead of size in bytesEstimated data
section are adjusted accordingly.The number of documents stored in the last 24 hours.
💡 Implementation hints
metering.index-info-task.enabled
should be enabled to be able to call the new endpoint (https://github.com/elastic/elasticsearch-serverless/pull/1753)The text was updated successfully, but these errors were encountered: