fix(@cubejs-client/react): options dependency for useEffect: check if…

… `subscribe` has been changed in `useCubeQuery` (#632)

Co-authored-by: Vasilev <aleksandr.vasiliev@elinvar.de>

packages/cubejs-client-react/src/useCubeQuery.js

@@ -4,6 +4,7 @@ import {
 import { equals } from 'ramda';
 import CubeContext from './CubeContext';
 import isQueryPresent from './isQueryPresent';
+import useDeepCompareMemoize from './useDeepCompareMemoize';
 
 export default (query, options = {}) => {
   const [mutexObj] = useState({});
@@ -65,7 +66,7 @@ export default (query, options = {}) => {
         subscribeRequest = null;
       }
     };
-  }, [JSON.stringify(query), options.cubejsApi, context]);
+  }, useDeepCompareMemoize([query, options, context]));
 
   return { isLoading, resultSet, error };
 };

packages/cubejs-client-react/src/useDeepCompareMemoize.js

@@ -0,0 +1,12 @@
+import { useRef } from 'react';
+import { equals } from 'ramda';
+
+export default function useDeepCompareMemoize(value) {
+  const ref = useRef([]);
+
+  if (!equals(value, ref.current)) {
+    ref.current = value;
+  }
+
+  return ref.current;
+}