feat: group by severity logs explorer page by default

[vikrantgupta25]

Summary

• the list view has by default group by of severity text for frequency chart.
• no change in any other existing behavviour

Related Issues / PR's

fixes #4727

Screenshots

Screen.Recording.2024-09-12.at.12.52.31.PM.mov

Affected Areas and Manually Tested Areas

• Logs Explorer Views Chart
• Live Logs Chart

[github-actions]

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

[github-actions]

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

[ankitnayan]

Please report the perf impact on scanning 50M or 100M rows

[vikrantgupta25]

Please report the perf impact on scanning 50M or 100M rows

https://www.notion.so/signoz/group-by-severity_text-78339feea9914f1c839f01f999904ad9 added the cases for 100M and 450M here.

[ankitnayan]

no legends for the logs explorer frequency chart as the colors are self explanatory.

This is only relevant when the groupby is severity_text. The legends are important if the groupby clause is something else like k8s.deployment.name

[ankitnayan]

I am still of the opinion of showing legends of different severity_text as there is no coloring convention and users might get confused with warning, debug and error, fatal categories.

[ankitnayan]

Also, a loom along with the PR helps a lot in seeing visual changes and 1st set of feedbacks without needing to run the PR

[vikrantgupta25]

This is only relevant when the groupby is severity_text. The legends are important if the groupby clause is something else like k8s.deployment.name

The frequency chart has a fixed group by of severity_text always hence no legends for the same.

I am still of the opinion of showing legends of different severity_text as there is no coloring convention and users might get confused with warning, debug and error, fatal categories.

We can do the same, also on the hover of the graph you will be able to see the details of label and the corresponding color.

Also, a loom along with the PR helps a lot in seeing visual changes and 1st set of feedbacks without needing to run the PR

yes i just removed the older video and was about to add the updated changes video soon before sharing it in channel 😅

[GeekBoySupreme]

Looks good to me @vikrantgupta25

[nityanandagohain]

The where clause and all the filters for the lower graph are picked and aggregateOperator and groupBy are overridden by Count and SeverityText respectively

will it help the user in some way when they are already doing groupby some other field ?
We can keep this functionality limited to the list view ?

[vikrantgupta25]

We can keep this functionality limited to the list view ?

done @nityanandagohain , added back the legends as well to make sure users don't get confused due to no standard color encodings.