Skip to content

Commit

Permalink
fix: include event params string and int value
Browse files Browse the repository at this point in the history
Reason: int value is needed for calculation of eng. rate and eng. time. String is added just in case and for some QA purposes
  • Loading branch information
RuslanBergenov committed Oct 11, 2024
1 parent 6f6252d commit 4f26e5d
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
2 changes: 2 additions & 0 deletions cf/main.py
Original file line number Diff line number Diff line change
Expand Up @@ -289,6 +289,8 @@ def get_event_params_query_select_statement(self):
CAST(event_params.value.float_value AS STRING),
CAST(event_params.value.double_value AS STRING)
) AS event_params_value,
event_params.value.string_value AS event_params_string_value,
event_params.value.int_value AS event_params_int_value,
{self.source_table_type} AS source_table_type
FROM
temp_events
Expand Down
2 changes: 2 additions & 0 deletions tests/rsc/sample_desired_queries.py
Original file line number Diff line number Diff line change
Expand Up @@ -128,6 +128,8 @@
CAST(event_params.value.float_value AS STRING),
CAST(event_params.value.double_value AS STRING)
) AS event_params_value,
event_params.value.string_value AS event_params_string_value,
event_params.value.int_value AS event_params_int_value,
'daily' AS source_table_type
Expand Down

0 comments on commit 4f26e5d

Please sign in to comment.