[logging] Add render failure error into log (apache#6422)

(cherry picked from commit 2489876)
graceguo-supercat · Nov 27, 2018 · 245a1ec · 245a1ec
1 parent 5a874f7
commit 245a1ec
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/superset/assets/src/chart/Chart.jsx b/superset/assets/src/chart/Chart.jsx
@@ -139,6 +139,14 @@ class Chart extends React.PureComponent {
         console.warn(error); // eslint-disable-line
         this.props.actions.chartRenderingFailed(error, this.props.chartId);
       });
+
+    Logger.append(LOG_ACTIONS_RENDER_CHART, {
+      slice_id: chartId,
+      has_err: true,
+      error_details: error.toString(),
+      start_offset: this.renderStartTime,
+      duration: Logger.getTimestamp() - this.renderStartTime,
+    });
   }
 
   addFilter(col, vals, merge = true, refresh = true) {