Use current_domain from each line (#102)

tinybirdco · Feb 6, 2025 · 4b633f3 · 4b633f3
1 parent 7d6a06b
commit 4b633f3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/tinybird/pipes/analytics_hits.pipe b/tinybird/pipes/analytics_hits.pipe
@@ -18,6 +18,7 @@ SQL >
         JSONExtractString(payload, 'referrer') as referrer,
         JSONExtractString(payload, 'pathname') as pathname,
         JSONExtractString(payload, 'href') as href,
+        domainWithoutWWW(href) as current_domain,
         lower(JSONExtractString(payload, 'user-agent')) as user_agent
     FROM analytics_events
     where action = 'page_hit'
@@ -33,6 +34,7 @@ SQL >
         referrer,
         pathname,
         href,
+        current_domain,
         case
             when match(user_agent, 'wget|ahrefsbot|curl|urllib|bitdiscovery|\+https://|googlebot')
             then 'bot'

diff --git a/tinybird/pipes/analytics_sources.pipe b/tinybird/pipes/analytics_sources.pipe
@@ -3,7 +3,6 @@ DESCRIPTION >
     Aggregate by referral and calculate session and hits
 
 SQL >
-    WITH (SELECT domainWithoutWWW(href) FROM analytics_hits LIMIT 1) AS currenct_domain
     SELECT
         toDate(timestamp) AS date,
         device,
@@ -13,7 +12,7 @@ SQL >
         uniqState(session_id) AS visits,
         countState() AS hits
     FROM analytics_hits
-    WHERE domainWithoutWWW(referrer) != currenct_domain
+    WHERE domainWithoutWWW(referrer) != current_domain
     GROUP BY date, device, browser, location, referrer
 
 TYPE MATERIALIZED