From 8fcd8ca594d0ebcb5ffe907eec998051452e6b4d Mon Sep 17 00:00:00 2001
From: Christopher Henn <chris@chrishenn.net>
Date: Mon, 7 Jan 2019 11:45:24 -0800
Subject: [PATCH] Use memoization in DygraphTransformation

---
 .../components/DygraphTransformation.tsx      | 30 +++++++------------
 1 file changed, 10 insertions(+), 20 deletions(-)

diff --git a/ui/src/shared/components/DygraphTransformation.tsx b/ui/src/shared/components/DygraphTransformation.tsx
index 1063a6b86c..22a321018c 100644
--- a/ui/src/shared/components/DygraphTransformation.tsx
+++ b/ui/src/shared/components/DygraphTransformation.tsx
@@ -1,7 +1,10 @@
 import {PureComponent} from 'react'
-import {FluxTable} from 'src/types'
+import memoizeOne from 'memoize-one'
+
 import {ErrorHandling} from 'src/shared/decorators/errors'
 
+import {FluxTable} from 'src/types'
+
 import {
   fluxTablesToDygraph,
   FluxTablesToDygraphResult,
@@ -13,27 +16,14 @@ interface Props {
 }
 
 @ErrorHandling
-class DygraphTransformation extends PureComponent<
-  Props,
-  FluxTablesToDygraphResult
-> {
-  public static getDerivedStateFromProps(props) {
-    return {...fluxTablesToDygraph(props.tables)}
-  }
-
-  constructor(props) {
-    super(props)
-
-    this.state = {
-      labels: [],
-      dygraphsData: [],
-      seriesDescriptions: [],
-    }
-  }
+class DygraphTransformation extends PureComponent<Props> {
+  private fluxTablesToDygraph = memoizeOne(fluxTablesToDygraph)
 
   public render() {
-    const {children} = this.props
-    return children(this.state)
+    const {tables, children} = this.props
+    const data = this.fluxTablesToDygraph(tables)
+
+    return children(data)
   }
 }