[GitHub] [superset] bkyryliuk commented on pull request #17782: ci: make presto hive tests to cover only chartData and sqljson

[GitBox <gi...@apache.org>]

bkyryliuk commented on pull request #17782:
URL: https://github.com/apache/superset/pull/17782#issuecomment-1006175139


   > i don't know the specific ones (not super familiar with the backend) so i'll stamp to unblock here. @bkyryliuk @villebro and @john-bodley would probably be the ones who know most about the backend to tag other tests. but this seems like a decent step (and could allow us to make these tests required for PRs too)
   
   just noticed the comment, I think we can enable presto / hive tests later on for the cases that are critical or if we will notice breakages. 


-- 
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: notifications-unsubscribe@superset.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org