superset-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [incubator-superset] john-bodley commented on pull request #11099: feat: more specific presto error messages
Date Tue, 29 Sep 2020 17:23:38 GMT

john-bodley commented on pull request #11099:
URL: https://github.com/apache/incubator-superset/pull/11099#issuecomment-700861841


   @etr2460 I think it depends (per your comments) on whether the errors are supposed to be
engine or Superset centric. If it’s the later then I think having universal errors makes
sense though the wording probably needs to be more aligned with the Superset vernacular.
   
   Note per the screenshots it messages do state "Presto Error" and thus it does currently
seem more engine centric.   


----------------------------------------------------------------
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:
users@infra.apache.org



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


Mime
View raw message