avoid parsing again on re-render in trace view

Signed-off-by: Shenoy Pratik <[email protected]>
opensearch-project · Feb 19, 2025 · 1c8e7e3 · 1c8e7e3
1 parent eed04fa
commit 1c8e7e3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/public/components/application_analytics/components/flyout_components/trace_detail_render.tsx b/public/components/application_analytics/components/flyout_components/trace_detail_render.tsx
@@ -38,7 +38,7 @@ export const TraceDetailRender = ({
   const [colorMap, setColorMap] = useState({});
 
   const storedFilters = sessionStorage.getItem('TraceAnalyticsSpanFilters');
-  const [spanFilters, setSpanFilters] = useState<TraceFilter[]>(
+  const [spanFilters, setSpanFilters] = useState<TraceFilter[]>(() =>
     storedFilters ? JSON.parse(storedFilters) : []
   );
 

diff --git a/public/components/trace_analytics/components/traces/trace_view.tsx b/public/components/trace_analytics/components/traces/trace_view.tsx
@@ -87,7 +87,7 @@ export function TraceView(props: TraceViewProps) {
   const [isGanttChartLoading, setIsGanttChartLoading] = useState(false);
 
   const storedFilters = sessionStorage.getItem('TraceAnalyticsSpanFilters');
-  const [spanFilters, setSpanFilters] = useState<TraceFilter[]>(
+  const [spanFilters, setSpanFilters] = useState<TraceFilter[]>(() =>
     storedFilters ? JSON.parse(storedFilters) : []
   );
   const [filteredPayload, setFilteredPayload] = useState('');