Merge pull request #71 from quickwit-oss/ddelemeny/no-query-trigger-onchange

Don't run query on editor change, fixes #70

Author: fmassot

src/components/QueryEditor/ElasticsearchQueryContext.test.tsx

@@ -41,7 +41,6 @@ describe('ElasticsearchQueryContext', () => {
     // Should also set timeField to the configured `timeField` option in datasource configuration
     expect(changedQuery.timeField).toBe(datasource.timeField);
 
-    expect(onRunQuery).toHaveBeenCalled();
   });
 
   // the following applies to all hooks in ElasticsearchQueryContext as they all share the same code.

src/components/QueryEditor/ElasticsearchQueryContext.tsx

@@ -41,9 +41,8 @@ export const ElasticsearchProvider = ({
   const onStateChange = useCallback(
     (query: ElasticsearchQuery) => {
       onChange(query);
-      onRunQuery();
     },
-    [onChange, onRunQuery]
+    [onChange]
   );
 
   const reducer = combineReducers<Pick<ElasticsearchQuery, 'query' | 'alias' | 'metrics' | 'bucketAggs'>>({