[GitHub] [incubator-druid] renevan10 commented on a change in pull request #8859: Web console: Fix missing include future flag for byPeriod rules

2019-11-12 Thread GitBox
renevan10 commented on a change in pull request #8859: Web console: Fix missing include future flag for byPeriod rules URL: https://github.com/apache/incubator-druid/pull/8859#discussion_r345497349 ## File path: web-console/src/views/datasource-view/datasource-view.tsx ##

[GitHub] [incubator-druid] renevan10 commented on a change in pull request #8859: Web console: Fix missing include future flag for byPeriod rules

2019-11-12 Thread GitBox
renevan10 commented on a change in pull request #8859: Web console: Fix missing include future flag for byPeriod rules URL: https://github.com/apache/incubator-druid/pull/8859#discussion_r345495745 ## File path: web-console/src/utils/load-rule.ts ## @@ -41,7 +42,13 @@ expo

[GitHub] [incubator-druid] renevan10 commented on a change in pull request #8859: Web console: Fix missing include future flag for byPeriod rules

2019-11-12 Thread GitBox
renevan10 commented on a change in pull request #8859: Web console: Fix missing include future flag for byPeriod rules URL: https://github.com/apache/incubator-druid/pull/8859#discussion_r345495728 ## File path: web-console/src/components/rule-editor/rule-editor.scss ## @@