jasonyanwenl commented on a change in pull request #5866:
URL: https://github.com/apache/incubator-pinot/pull/5866#discussion_r473375909



##########
File path: thirdeye/thirdeye-pinot/src/main/resources/schema/create-schema.sql
##########
@@ -437,3 +437,15 @@ create index rootcause_template_id_idx ON 
rootcause_template_index(base_id);
 create index rootcause_template_owner_idx ON rootcause_template_index(owner);
 create index rootcause_template_metric_idx on 
rootcause_template_index(metric_id);
 create index rootcause_template_config_application_idx ON 
rootcause_template_index(`application`);
+
+create table if not exists online_detection_data_index (
+    base_id bigint(20) not null,
+    dataset varchar(200),
+    metric varchar(200),
+    create_time timestamp default 0,
+    update_time timestamp default current_timestamp,
+    version int(10)
+) ENGINE=InnoDB;

Review comment:
       Thanks for this! I agree with you. In long term, we should use a more 
efficient way to store the data. This is just current workaround. And if we use 
a type to store it such as the BLOB, we cannot store it into the 
`generic_json_entity` table and need to consider where to store it.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to