refactor 'Online Partitions' column

Author: bachmanity1

frontend/src/components/Brokers/BrokersList/TableCells/TableCells.tsx

@@ -72,23 +72,3 @@ export const Skew = ({ getValue }: ScewProps) => {
     />
   );
 };
-
-type OnlinePartitionsProps = CellContext<
-  BrokersTableRow,
-  BrokersTableRow['inSyncPartitions']
->;
-
-export const OnlinePartitions = ({ row }: OnlinePartitionsProps) => {
-  const { count, inSyncPartitions } = row.original;
-
-  if (count === undefined || inSyncPartitions === undefined) {
-    return null;
-  }
-
-  return (
-    <ColoredCell
-      value={inSyncPartitions}
-      attention={inSyncPartitions !== count}
-    />
-  );
-};

frontend/src/components/Brokers/BrokersList/lib/types.ts

@@ -7,7 +7,7 @@ export type BrokersTableRow = {
   partitionsLeader: number | undefined;
   partitionsSkew: number | undefined;
   leadersSkew: number | undefined;
-  inSyncPartitions: number | undefined;
+  partitions: number | undefined;
   onlinePartitionCount: number | undefined;
   offlinePartitionCount: number | undefined;
   activeControllers: number | undefined;

frontend/src/components/Brokers/BrokersList/lib/utils.ts

@@ -40,7 +40,7 @@ export const getBrokersTableRows = ({
       partitionsLeader: broker.partitionsLeader,
       partitionsSkew: broker.partitionsSkew,
       leadersSkew: broker.leadersSkew,
-      inSyncPartitions: broker.inSyncPartitions,
+      partitions: broker.partitions,
       onlinePartitionCount,
       offlinePartitionCount,
       activeControllers,
@@ -60,19 +60,16 @@ export const getBrokersTableColumns = () => {
       header: 'Disk usage',
       cell: Cell.DiscUsage,
     }),
+    columnHelper.accessor('partitions', { header: 'Partitions' }),
     columnHelper.accessor('partitionsSkew', {
       header: SkewHeader,
       cell: Cell.Skew,
     }),
     columnHelper.accessor('partitionsLeader', { header: 'Leaders' }),
     columnHelper.accessor('leadersSkew', {
-      header: 'Leader skew',
+      header: 'Leaders skew',
       cell: Cell.Skew,
     }),
-    columnHelper.accessor('inSyncPartitions', {
-      header: 'Online partitions',
-      cell: Cell.OnlinePartitions,
-    }),
     columnHelper.accessor('port', { header: 'Port' }),
     columnHelper.accessor('host', { header: 'Host' }),
   ];