tangdian commented on a change in pull request #3830: [TE] SQL Connector 
backend and front end, supporting Presto, MySQL, H2, with sample data in H2
URL: https://github.com/apache/incubator-pinot/pull/3830#discussion_r271814435
 
 

 ##########
 File path: 
thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/datasource/mock/AutoOnboardMockDataSource.java
 ##########
 @@ -61,16 +61,6 @@ public AutoOnboardMockDataSource(MetadataSourceConfig 
metadataSourceConfig) {
 
   @Override
   public void run() {
-    if (!this.datasetDAO.findAll().isEmpty()) {
 
 Review comment:
   I'm not sure. Let me check with Xiaohui.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to