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 2019/11/27 00:33:59 UTC
[GitHub] [incubator-superset] etr2460 opened a new issue #8670:
presto.latest_sub_partition fails when table contains column named
`table_name`
etr2460 opened a new issue #8670: presto.latest_sub_partition fails when table contains column named `table_name`
URL: https://github.com/apache/incubator-superset/issues/8670
presto.latest_sub_partition fails when table contains column named `table_name`
### Expected results
the template to work
### Actual results
it fails
#### How to reproduce the bug
1. Create a presto table containing a column named `table_name`
2. Try to use latest_sub_partition and see it fail
### Environment
(please complete the following information):
- superset version: `master`
- python version: `3.6`
- node.js version: `8.6`
### Checklist
Make sure these boxes are checked before submitting your issue - thank you!
- [x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
- [x] I have reproduced the issue with at least the latest released version of superset.
- [x] I have checked the issue tracker for the same issue and I haven't found one similar.
### Additional context
We should rename the named arguments for any template functions like this so that they don't conflict with columns in the table themselves. Probably by creating a very long variable name that's unlikely to be a column name for a table
----------------------------------------------------------------
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
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org