address some coderabbit nitpicks

Author: sosiouxme

pkg/filter/filterable.go

@@ -244,6 +244,8 @@ func (f FilterItem) toBQStr(filterable Filterable, paramIndex int) (sql string,
 	makeNumParam := func() []bigquery.QueryParameter {
 		num, err := strconv.ParseFloat(f.Value, 64)
 		if err != nil {
+			log.Errorf("Failed to parse numeric filter value %q for field %s: %v", f.Value, f.Field, err)
+
 			return makeParam("NOT A NUMBER: " + f.Value) // which will break appropriately
 		}
 		return makeParam(num)
@@ -306,6 +308,7 @@ func (f FilterItem) toBQStr(filterable Filterable, paramIndex int) (sql string,
 		return fmt.Sprintf("%s IS %s NULL", field, optNot(!f.Not)), nil
 	}
 
+	log.Errorf("Unknown filter operator %q for field %s", f.Operator, f.Field)
 	return "UnknownFilterOperator()", nil // cause SQL to fail in obvious way
 }
 

sippy-ng/src/jobs/JobRunsTable.js

@@ -361,6 +361,9 @@ export default function JobRunsTable(props) {
     fetch(process.env.REACT_APP_API_URL + '/api/labels')
       .then((response) => {
         if (response.status !== 200) {
+          console.warn(
+            'Labels API returned unsuccessful status: ' + response.statusText
+          )
           return {}
         }
         return response.json()