villebro commented on issue #9572: URL: https://github.com/apache/incubator-superset/pull/9572#issuecomment-616340973
Looking more closely at the default jinja functions, it appears they should all default to being added to extra cache keys, except `filter_values` and `form_data`: - `url_param` - `current_user_id` - `current_username` Then regular users wouldn't need to concern themselves with wrapping the function calls, and `cache_key_wrapper` would be something that mostly more advanced users need when introducing custom jinja functions. ---------------------------------------------------------------- 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]
