eschutho commented on PR #19981:
URL: https://github.com/apache/superset/pull/19981#issuecomment-1129419083

   > @eschutho I'm a little lost, why do we need the `datasource_type` post the 
Druid NoSQL deprecation? i.e., this served merely to ensure the 
(`datasource_type`, `datasouce_id`) tuple was unique but given we only now have 
`datasource_type = 'table` this is irrelevant isn't it—unless this is somehow 
being used @betodealmeida's table/query schema refactor.
   
   yeah, this is a precursor for 
https://github.com/apache/superset/issues/19953 where we'll be introducing 
other datasource types: tables, queries, savedqueries. 


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


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

Reply via email to