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

2019-11-12 Thread GitBox
vogievetsky 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_r345501797 ## File path: web-console/src/views/datasource-view/datasource-view.tsx ##

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

2019-11-12 Thread GitBox
vogievetsky 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_r345476443 ## File path: web-console/src/utils/load-rule.ts ## @@ -41,7 +42,13 @@ ex

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

2019-11-12 Thread GitBox
vogievetsky 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_r345475987 ## File path: web-console/src/components/rule-editor/rule-editor.scss ##

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

2019-11-12 Thread GitBox
vogievetsky 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_r345476663 ## File path: web-console/src/views/datasource-view/datasource-view.tsx ##