You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2020/09/29 17:23:38 UTC

[GitHub] [incubator-superset] john-bodley commented on pull request #11099: feat: more specific presto error messages

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