junlincc edited a comment on pull request #14485:
URL: https://github.com/apache/superset/pull/14485#issuecomment-835121290


   looks like we did some engineering QA before approving and merging this PR, 
i'm curious to know how the product could be affected in general by 
migration/refractor PR like this one? how can we better catch regression going 
forward? the regression can be seen in all charts that have Group-by select.  
   
   @villebro @rusackas @jinghua-qa 


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



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

Reply via email to