Revert "Bug fixes for observability count distribution and applicatio…

…n analytics (opensearch-project#1187) (opensearch-project#1189)" This reverts commit e2440d2.
mengweieric · Nov 8, 2023 · 1f63d69 · 1f63d69
1 parent 7e4693d
commit 1f63d69
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 20 deletions.
diff --git a/public/components/common/query_utils/index.ts b/public/components/common/query_utils/index.ts
@@ -245,10 +245,7 @@ export const buildPatternsQuery = (
   return finalQuery;
 };
 
-export const buildQuery = (baseQuery: string, currQuery: string) => {
-  if (!currQuery) return baseQuery;
-  return `${baseQuery} | ${currQuery}`;
-};
+export const buildQuery = (baseQuery: string, currQuery: string) => baseQuery + '| ' + currQuery;
 
 export const buildRawQuery = (query: IQuery, appBaseQuery: string) => {
   if (appBaseQuery && !query.rawQuery.includes(appBaseQuery))

diff --git a/public/components/event_analytics/explorer/explorer.tsx b/public/components/event_analytics/explorer/explorer.tsx
@@ -258,12 +258,6 @@ export const Explorer = ({
       ...TIME_INTERVAL_OPTIONS,
     ]);
     selectedIntervalRef.current = { text: 'Auto', value: 'auto_' + minInterval };
-    dispatch(
-      updateCountDistribution({
-        tabId,
-        data: { selectedInterval: selectedIntervalRef.current.value.replace(/^auto_/, '') },
-      })
-    );
   };
 
   useEffect(() => {
@@ -501,18 +495,13 @@ export const Explorer = ({
                           selectedIntervalRef.current = timeIntervalOptions[intervalOptionsIndex];
                           getPatterns(intrv, getErrorHandler('Error fetching patterns'));
                         }}
-                        stateInterval={
-                          countDistribution.selectedInterval || selectedIntervalRef.current?.value
-                        }
+                        stateInterval={selectedIntervalRef.current?.value}
                         startTime={appLogEvents ? startTime : dateRange[0]}
                         endTime={appLogEvents ? endTime : dateRange[1]}
                       />
-                      <EuiSpacer size="s" />
                       <CountDistribution
                         countDistribution={countDistribution}
-                        selectedInterval={
-                          countDistribution.selectedInterval || selectedIntervalRef.current?.value
-                        }
+                        selectedInterval={selectedIntervalRef.current?.value}
                         startTime={appLogEvents ? startTime : dateRange[0]}
                         endTime={appLogEvents ? endTime : dateRange[1]}
                       />
@@ -794,8 +783,6 @@ export const Explorer = ({
     subType,
     selectedCustomPanelOptions,
     explorerSearchMeta,
-    selectedIntervalRef.current,
-    countDistribution,
   ]);
 
   const liveTailLoop = async (

diff --git a/...ponents/event_analytics/explorer/visualizations/count_distribution/count_distribution.tsx b/...ponents/event_analytics/explorer/visualizations/count_distribution/count_distribution.tsx
@@ -4,6 +4,7 @@
  */
 
 import React from 'react';
+import { EuiPanel } from '@elastic/eui';
 import { BarOrientation, LONG_CHART_COLOR } from '../../../../../../common/constants/shared';
 import { Plt } from '../../../../visualizations/plotly/plot';
 import { fillTimeDataWithEmpty } from '../../../utils/utils';

diff --git a/public/components/event_analytics/redux/slices/count_distribution_slice.ts b/public/components/event_analytics/redux/slices/count_distribution_slice.ts
@@ -17,7 +17,6 @@ export const countDistributionSlice = createSlice({
   reducers: {
     render: (state, { payload }) => {
       state[payload.tabId] = {
-        ...state[payload.tabId],
         ...payload.data,
       };
     },