opensearch-project / dashboards-observability

Visualize and explore your logs, traces and metrics data in OpenSearch Dashboards
https://opensearch.org/docs/latest/observability-plugin/index/
Apache License 2.0
14 stars 46 forks source link

Refactor of Trace analytics to remove duplicated checks. #1920

Open TackAdam opened 1 week ago

TackAdam commented 1 week ago

Refactor of Trace analytics to improve readability by reducing duplicated checks.

Description

Files/calls using <mode === 'data_prepper'> and <mode === 'jaeger'> multiple times replaced with isDataPrepper and isJaeger to improve readability.

Also removed currPage argument from onTableChange in /trace_analytics/components/traces/traces_table.tsx call as it was not used in the function and was throwing an error.

Check List

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. For more information on following Developer Certificate of Origin and signing off your commits, please check here.