You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by el...@apache.org on 2024/04/16 22:18:29 UTC

(superset) branch elizabeth/test-thumbnails created (now b454f422d8)

This is an automated email from the ASF dual-hosted git repository.

elizabeth pushed a change to branch elizabeth/test-thumbnails
in repository https://gitbox.apache.org/repos/asf/superset.git


      at b454f422d8 add username logging to thumbnails

This branch includes the following new commits:

     new b454f422d8 add username logging to thumbnails

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.



(superset) 01/01: add username logging to thumbnails

Posted by el...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

elizabeth pushed a commit to branch elizabeth/test-thumbnails
in repository https://gitbox.apache.org/repos/asf/superset.git

commit b454f422d87b49cb5dfec935101c484a6cfa17a1
Author: Elizabeth Thompson <es...@gmail.com>
AuthorDate: Tue Apr 16 15:18:19 2024 -0700

    add username logging to thumbnails
---
 superset/tasks/thumbnails.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/superset/tasks/thumbnails.py b/superset/tasks/thumbnails.py
index 03b3999dce..c8daa069b0 100644
--- a/superset/tasks/thumbnails.py
+++ b/superset/tasks/thumbnails.py
@@ -52,12 +52,12 @@ def cache_chart_thumbnail(
         logger.warning("No chart found, skip computing chart thumbnail")
         return None
     url = get_url_path("Superset.slice", slice_id=chart.id)
-    logger.info("Caching chart: %s", url)
     _, username = get_executor(
         executor_types=current_app.config["THUMBNAIL_EXECUTE_AS"],
         model=chart,
         current_user=current_user,
     )
+    logger.info("Caching chart: %s as %s", url, username)
     user = security_manager.find_user(username)
     with override_user(user):
         screenshot = ChartScreenshot(url, chart.digest)