Merge pull request #129 from quickwit-oss/ddelemeny/fix-context-varia…

…ble-queries Disable lint filter in context editor
quickwit-oss · May 16, 2024 · e88acdb · e88acdb
2 parents 6628ce2 + 9f9a2ff
commit e88acdb
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/src/LogContext/components/LogContextUI.tsx b/src/LogContext/components/LogContextUI.tsx
@@ -49,8 +49,8 @@ export interface LogContextUIProps extends LogContextProps {
 
 export function LogContextUI(props: LogContextUIProps ){
   const builder = useQueryBuilder();
-  const {query, parsedQuery, setQuery, setParsedQuery} = builder;
-  const [canRunQuery, setCanRunQuery] = useState<boolean>(false);
+  const {query, /*parsedQuery,*/ setQuery, setParsedQuery} = builder;
+  const [canRunQuery, /*setCanRunQuery*/] = useState<boolean>(true);
   const {row, origQuery, updateQuery, runContextQuery } = props;
 
   const fieldsSuggestionTimeRange = useMemo(()=>createContextTimeRange(row.timeEpochMs), [row])
@@ -60,9 +60,11 @@ export function LogContextUI(props: LogContextUIProps ){
     setQuery(origQuery?.query || '')
   }, [setQuery, origQuery])
 
-  useEffect(()=>{
-    setCanRunQuery(!parsedQuery.parseError)
-  }, [parsedQuery, setCanRunQuery])
+  // FIXME : query parser used for lint is not reliable enough
+  // to use as a filter for wrong queries. Disabled for now.
+  // useEffect(()=>{
+  //   <setCanRunQuery(!parsedQuery.parseError)
+  // }, [parsedQuery, setCanRunQuery])
 
   const runQuery = useCallback(()=>{
     if (runContextQuery){