Merge pull request #38 from fishtown-analytics/fix/null-session-number

Claire Carroll · web-flow · commit a6914e8ee30f · 2020-02-25T10:43:59.000-08:00
Fix null session numbers
diff --git a/macros/cross-adapter-modeling/sessionization/segment_web_sessions.sql b/macros/cross-adapter-modeling/sessionization/segment_web_sessions.sql
@@ -72,7 +72,7 @@ windowed as (
             partition by blended_user_id
             order by sessions.session_start_tstamp
             )
-            {% if is_incremental() %}+ agg.starting_session_number {% endif %}
+            {% if is_incremental() %}+ coalesce(agg.starting_session_number, 0) {% endif %}
             as session_number
 
     from sessions

Original file line number	Diff line number	Diff line change
`@@ -72,7 +72,7 @@ windowed as (`
`72`	`72`	`partition by blended_user_id`
`73`	`73`	`order by sessions.session_start_tstamp`
`74`	`74`	`)`
`75`		`- {% if is_incremental() %}+ agg.starting_session_number {% endif %}`
	`75`	`+ {% if is_incremental() %}+ coalesce(agg.starting_session_number, 0) {% endif %}`
`76`	`76`	`as session_number`
`77`	`77`
`78`	`78`	`from sessions`