fixes: sorting, clean up side effect hooks

questdb · Dec 2, 2024 · 9b16745 · 9b16745
1 parent 1ea0d14
commit 9b16745
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 15 deletions.
diff --git a/packages/browser-tests/questdb b/packages/browser-tests/questdb
diff --git a/packages/web-console/src/scenes/Editor/Metrics/metric.tsx b/packages/web-console/src/scenes/Editor/Metrics/metric.tsx
@@ -216,20 +216,11 @@ export const Metric = ({
 
   useEffect(() => {
     metricDurationRef.current = metricDuration
+
     if (metric.tableId) {
       fetchMetric()
       setupListeners()
     }
-  }, [metricDuration, metric.tableId])
-
-  const focusListener = useCallback(() => {
-    if (focusListenerRef.current) {
-      fetchMetric()
-    }
-  }, [metric.tableId])
-
-  useEffect(() => {
-    setupListeners()
 
     return () => {
       if (intervalRef.current) {
@@ -240,7 +231,13 @@ export const Metric = ({
         focusListenerRef.current = false
       }
     }
-  }, [autoRefreshTables])
+  }, [autoRefreshTables, metricDuration, metric.tableId])
+
+  const focusListener = useCallback(() => {
+    if (focusListenerRef.current) {
+      fetchMetric()
+    }
+  }, [metric.tableId])
 
   if (!data && !loading && metric.tableId)
     return (

diff --git a/packages/web-console/src/scenes/Editor/Metrics/table-selector.tsx b/packages/web-console/src/scenes/Editor/Metrics/table-selector.tsx
@@ -98,9 +98,11 @@ export const TableSelector = ({
   const enterPress = useKeyPress("Enter")
   const wrapperRef = useRef<HTMLDivElement | null>(null)
 
-  const filteredOptions = options.filter((option) =>
-    query ? option.label.toLowerCase().includes(query.toLowerCase()) : true,
-  )
+  const filteredOptions = options
+    .filter((option) =>
+      query ? option.label.toLowerCase().includes(query.toLowerCase()) : true,
+    )
+    .sort((a, b) => a.label.toLowerCase().localeCompare(b.label.toLowerCase()))
 
   useEffect(() => {
     if (!inputRef.current) return

diff --git a/packages/web-console/src/scenes/Schema/index.tsx b/packages/web-console/src/scenes/Schema/index.tsx
@@ -347,6 +347,11 @@ const Schema = ({
                   : true)
               )
             })
+            .sort((a, b) =>
+              a.table_name
+                .toLowerCase()
+                .localeCompare(b.table_name.toLowerCase()),
+            )
             .map((table: QuestDB.Table) => (
               <Table
                 designatedTimestamp={table.designatedTimestamp}
+1 −1		core/src/main/java/io/questdb/cutlass/pgwire/PGConnectionContext.java
+43 −0		core/src/main/java/io/questdb/cutlass/pgwire/modern/PGConnectionContextModern.java
+54 −24		core/src/main/java/io/questdb/cutlass/pgwire/modern/PGPipelineEntry.java
+53 −1		core/src/test/java/io/questdb/test/cutlass/pgwire/PGJobContextTest.java
+50 −0		core/src/test/java/io/questdb/test/cutlass/pgwire/PGTlsCompatTest.java
+4 −4		core/src/test/java/io/questdb/test/cutlass/pgwire/PgBootstrapTest.java