remove unused timers page

commafeed-client/src/pages/admin/MetricsPage.tsx

@@ -1,13 +1,11 @@
-import { Accordion, Box, Tabs } from "@mantine/core"
+import { Accordion, Box } from "@mantine/core"
 import { client } from "app/client"
 import { Loader } from "components/Loader"
 import { Gauge } from "components/metrics/Gauge"
 import { Meter } from "components/metrics/Meter"
 import { MetricAccordionItem } from "components/metrics/MetricAccordionItem"
-import { Timer } from "components/metrics/Timer"
 import { useEffect } from "react"
 import { useAsync } from "react-async-hook"
-import { TbChartAreaLine, TbClock } from "react-icons/tb"
 
 const shownMeters: Record<string, string> = {
     "com.commafeed.backend.feed.FeedRefreshEngine.refill": "Feed queue refill rate",
@@ -42,46 +40,25 @@ export function MetricsPage() {
     }, [query.execute])
 
     if (!query.result) return <Loader />
-    const { meters, gauges, timers } = query.result.data
+    const { meters, gauges } = query.result.data
     return (
-        <Tabs defaultValue="stats">
-            <Tabs.List>
-                <Tabs.Tab value="stats" leftSection={<TbChartAreaLine size={14} />}>
-                    Stats
-                </Tabs.Tab>
-                <Tabs.Tab value="timers" leftSection={<TbClock size={14} />}>
-                    Timers
-                </Tabs.Tab>
-            </Tabs.List>
+        <>
+            <Accordion variant="contained" chevronPosition="left">
+                {Object.keys(shownMeters).map(m => (
+                    <MetricAccordionItem key={m} metricKey={m} name={shownMeters[m]} headerValue={meters[m].count}>
+                        <Meter meter={meters[m]} />
+                    </MetricAccordionItem>
+                ))}
+            </Accordion>
 
-            <Tabs.Panel value="stats" pt="xs">
-                <Accordion variant="contained" chevronPosition="left">
-                    {Object.keys(shownMeters).map(m => (
-                        <MetricAccordionItem key={m} metricKey={m} name={shownMeters[m]} headerValue={meters[m].count}>
-                            <Meter meter={meters[m]} />
-                        </MetricAccordionItem>
-                    ))}
-                </Accordion>
-
-                <Box pt="xs">
-                    {Object.keys(shownGauges).map(g => (
-                        <Box key={g}>
-                            <span>{shownGauges[g]}:&nbsp;</span>
-                            <Gauge gauge={gauges[g]} />
-                        </Box>
-                    ))}
-                </Box>
-            </Tabs.Panel>
-
-            <Tabs.Panel value="timers" pt="xs">
-                <Accordion variant="contained" chevronPosition="left">
-                    {Object.keys(timers).map(key => (
-                        <MetricAccordionItem key={key} metricKey={key} name={key} headerValue={timers[key].count}>
-                            <Timer timer={timers[key]} />
-                        </MetricAccordionItem>
-                    ))}
-                </Accordion>
-            </Tabs.Panel>
-        </Tabs>
+            <Box pt="xs">
+                {Object.keys(shownGauges).map(g => (
+                    <Box key={g}>
+                        <span>{shownGauges[g]}:&nbsp;</span>
+                        <Gauge gauge={gauges[g]} />
+                    </Box>
+                ))}
+            </Box>
+        </>
     )
 }

commafeed-server/TODO.md

@@ -18,5 +18,3 @@ Nice to have:
 - rename "servlets" since they are now rest endpoints
 - warnings hibernate on startup
 - OPML encoding is not handled correctly
-- remove Timers metrics page
-