Adding ability to install re_data for Greenplum database #102
Conversation
| @@ -6,7 +6,7 @@ with median_value as ( | |||
| column_name, | |||
| metric, | |||
| interval_length_sec, | |||
| avg(value) {% if target.type != 'postgres' %} over(partition by {{ columns_to_group_by }}) {% endif %} as last_avg, | |||
There was a problem hiding this comment.
To not repeat and make it more extensible, I think macro with those 2 db would be nice. So that we could write something like this:
{% if target.type in postgres_type_db() %}
There was a problem hiding this comment.
Agree, sounds reasonable. Do you have any recommendations where to place this macro?
For me macros/db/postgres is the best folder for it.
|
@mrprigun Thanks for the PR! I've added one comment regarding implementation (suggestion on making it more extensible for the future). I believe |
1 similar comment
|
@mrprigun Thanks for the PR! I've added one comment regarding implementation (suggestion on making it more extensible for the future). I believe |
What
Fixing [BUG] Unable to install re_data for Greenplum database #100
to_single_json.sqlfile formattingHow
Fixed related Jinja SQL templates