Remove unecessary async

elastic · Jun 6, 2024 · 06e70bf · 06e70bf
1 parent 210cfaf
commit 06e70bf
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/plugins/data/public/actions/filters/create_filters_from_value_click.ts b/src/plugins/data/public/actions/filters/create_filters_from_value_click.ts
@@ -158,13 +158,13 @@ export const createFiltersFromValueClickAction = async ({
 };
 
 /** @public */
-export const appendFilterToESQLQueryFromValueClickAction = async ({
+export const appendFilterToESQLQueryFromValueClickAction = ({
   data,
   query,
 }: ValueClickDataContext) => {
   let queryWithFilter: string | undefined;
 
-  await Promise.all(
+  Promise.all(
     data
       .filter((point) => point)
       .map(async (val) => {

diff --git a/src/plugins/data/public/actions/value_click_action.ts b/src/plugins/data/public/actions/value_click_action.ts
@@ -55,7 +55,7 @@ export function createValueClickActionDefinition(
         if (context.data.query && isOfAggregateQueryType(context.data.query)) {
           // ES|QL charts have a different way of applying filters,
           // they are appending a where clause to the query
-          const queryString = await appendFilterToESQLQueryFromValueClickAction(context.data);
+          const queryString = appendFilterToESQLQueryFromValueClickAction(context.data);
           await getStartServices().uiActions.getTrigger('UPDATE_ESQL_QUERY_TRIGGER').exec({
             embeddable: context.embeddable,
             queryString,