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

Telemetry: TOTAL_INDICES_SIZE need to be a Numeric Metric #28316

Open
freddyDOTCMS opened this issue Apr 22, 2024 · 0 comments
Open

Telemetry: TOTAL_INDICES_SIZE need to be a Numeric Metric #28316

freddyDOTCMS opened this issue Apr 22, 2024 · 0 comments

Comments

@freddyDOTCMS
Copy link
Contributor

freddyDOTCMS commented Apr 22, 2024

Parent Issue

No response

Problem Statement

The TOTAL_INDICES_SIZE is a Metric to return the sum of all the Site Search indices, right now this metric is returning the value as follow: "4.6mb", "1.8gb" or "0b", this is not a numeric value and this is preventing us from run query filtering by the size of the indices that the client are using. For example:

Listing all the client that are using more than 2gb on their Site Search indices.

To turn it as a numeric value so we need to return it on the same unit all the time (maybe mb is the best option)

Also The Metric Snapshot right now has the follow sintax:

{
  stats: [
      { 
         feature :  
         description : 
         category :
         name : 
         value : 
      }
      ...
   ],
   no_numeric_metric_1: [value],
   no_numeric_metric_2: [value],
   ... 
  no_numeric_metric_1: [value] 
}

Inside the stats array are list the numeric Metric and outside are the no numeric metrics, after this change the TOTAL_INDICES_SIZE Metric need to be list inside the stats array on this way this is going to be handle as a Numeric Metric on all the SQL Query that we are using.

Steps to Reproduce

Check any Snapshot field on the PROD Telemetry data base

Acceptance Criteria

  • The TOTAL_INDICES_SIZE must be a numeric value and be list on the stats array
  • The TOTAL_INDICES_SIZE must return 0 when does not exists any Site Search Index.
  • The TOTAL_INDICES_SIZE must return the Size of the Index when exists just one Site Search Index.
  • The TOTAL_INDICES_SIZE must return the Sum of the Size of the Index when exists more than one Site Search Index.

The Value must be represent on the Same Unit all the time, it means can not be return as mb sometime and other as gb.
I think the best approched here is use mb

dotCMS Version

latest version of the telemetry plugin

Proposed Objective

Core Features

Proposed Priority

Priority 3 - Average

External Links... Slack Conversations, Support Tickets, Figma Designs, etc.

No response

Assumptions & Initiation Needs

No response

Quality Assurance Notes & Workarounds

No response

Sub-Tasks & Estimates

No response

@freddyDOTCMS freddyDOTCMS changed the title Telemetry: TOTAL_INDICES_SIZE Metric is not located correctly Telemetry: TOTAL_INDICES_SIZE need to be a Numeric Metric Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Future
Development

No branches or pull requests

2 participants