From dbdbb5ff259ee4d4e6c229eeb28bc7016e01ecbb Mon Sep 17 00:00:00 2001
From: Jc <jcw-@users.noreply.github.com>
Date: Tue, 28 Jul 2020 14:08:04 -0600
Subject: [PATCH] fix: allow renewQuery in dev mode with warning (#868) Thanks
 to @jcw-!

* fix: allow renewQuery in dev mode with warning

* Empty commit to trigger a CI rerun

* Fix specs
---
 .../cubejs-playground/src/components/CachePane.js     |  1 +
 .../orchestrator/PreAggregations.js                   | 11 ++++++++++-
 2 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/packages/cubejs-playground/src/components/CachePane.js b/packages/cubejs-playground/src/components/CachePane.js
index f521a2c3aa645..1ef4d0f05987f 100644
--- a/packages/cubejs-playground/src/components/CachePane.js
+++ b/packages/cubejs-playground/src/components/CachePane.js
@@ -76,6 +76,7 @@ const CachePane = ({ query, cubejsApi }) => (
                   title: 'Refresh Key Value',
                   key: 'value',
                   render: (text, record) => rs && rs.loadResponse.usedPreAggregations
+                    && rs.loadResponse.usedPreAggregations[record.tableName].refreshKeyValues
                     && rs.loadResponse.usedPreAggregations[record.tableName].refreshKeyValues.map(k => (
                       <PrismCode
                         key={JSON.stringify(k)}
diff --git a/packages/cubejs-query-orchestrator/orchestrator/PreAggregations.js b/packages/cubejs-query-orchestrator/orchestrator/PreAggregations.js
index 228251e2efec0..132520fa534ff 100644
--- a/packages/cubejs-query-orchestrator/orchestrator/PreAggregations.js
+++ b/packages/cubejs-query-orchestrator/orchestrator/PreAggregations.js
@@ -182,7 +182,16 @@ class PreAggregationLoader {
     this.structureVersionPersistTime = preAggregations.structureVersionPersistTime;
     this.externalRefresh = options.externalRefresh;
     if (this.externalRefresh && this.waitForRenew) {
-      throw new Error("Invalid configuration - when externalRefresh is true, it will not perform a renew, therefore you cannot wait for it using waitForRenew.");
+      const message = 'Invalid configuration - when externalRefresh is true, it will not perform a renew, therefore you cannot wait for it using waitForRenew.';
+      if (['production', 'test'].includes(process.env.NODE_ENV)) {
+        throw new Error(message);
+      } else {
+        this.logger('Invalid Configuration', {
+          requestId: this.requestId,
+          warning: message,
+        });
+        this.waitForRenew = false;
+      }
     }
   }