mistercrunch closed pull request #4645: CRUD hints around SQL expressions URL: https://github.com/apache/incubator-superset/pull/4645
This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/superset/connectors/sqla/views.py b/superset/connectors/sqla/views.py index 8398cbcbdd..b667475033 100644 --- a/superset/connectors/sqla/views.py +++ b/superset/connectors/sqla/views.py @@ -57,8 +57,8 @@ class TableColumnInlineView(CompactCRUDMixin, SupersetModelView): # noqa 'expression-defined columns in some cases. In most case ' 'users should not need to alter this.'), 'expression': utils.markdown( - 'a valid SQL expression as supported by the underlying backend. ' - 'Example: `substr(name, 1, 1)`', True), + 'a valid, *non-aggregating* SQL expression as supported by the ' + 'underlying backend. Example: `substr(name, 1, 1)`', True), 'python_date_format': utils.markdown(Markup( 'The pattern of timestamp format, use ' '<a href="https://docs.python.org/2/library/' @@ -114,8 +114,8 @@ class SqlMetricInlineView(CompactCRUDMixin, SupersetModelView): # noqa 'expression', 'table', 'd3format', 'is_restricted', 'warning_text'] description_columns = { 'expression': utils.markdown( - 'a valid SQL expression as supported by the underlying backend. ' - 'Example: `count(DISTINCT userid)`', True), + 'a valid, *aggregating* SQL expression as supported by the ' + 'underlying backend. Example: `count(DISTINCT userid)`', True), 'is_restricted': _('Whether the access to this metric is restricted ' 'to certain roles. Only roles with the permission ' "'metric access on XXX (the name of this metric)' " ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services