bugfix: deduplicate account map (#417)

Thanks to  Andrea Caldarone for the SQL Optimization

Author: iakov-aws

cid/builtin/core/data/queries/shared/account_map_dummy.sql

@@ -0,0 +1,10 @@
+CREATE OR REPLACE VIEW  ${athena_view_name} AS
+SELECT DISTINCT
+    line_item_usage_account_id                                     account_id,
+    MAX_BY(bill_payer_account_id,      line_item_usage_start_date) parent_account_id,
+    MAX_BY(line_item_usage_account_id, line_item_usage_start_date) account_name,
+    MAX_BY(line_item_usage_account_id, line_item_usage_start_date) account_email_id
+FROM
+    "${cur_table_name}"
+GROUP BY
+    line_item_usage_account_id

cid/helpers/account_map.py

@@ -156,24 +156,16 @@ def create(self, name) -> bool:
     def get_dummy_account_mapping_sql(self, name) -> list:
         """Create dummy account mapping"""
         logger.info(f'Creating dummy account mapping for {name}')
-        template_str = '''CREATE OR REPLACE VIEW  ${athena_view_name} AS SELECT DISTINCT
-            line_item_usage_account_id account_id, bill_payer_account_id parent_account_id,
-            line_item_usage_account_id account_name, line_item_usage_account_id account_email_id
-            FROM
-                "${cur_table_name}"
-        '''
-        template = Template(template_str)        
-        # Fill in TPLs
-        columns_tpl = dict()
-        parameters = {
+        template = Template(resource_string(
+            package_or_requirement='cid.builtin.core',
+            resource_name='data/queries/shared/account_map_dummy.sql',
+        ).decode('utf-8'))
+        columns_tpl = {
             'athena_view_name': name,
             'cur_table_name': self.cur.tableName
         }
-        columns_tpl.update(**parameters)
         compiled_query = template.safe_substitute(columns_tpl)
-        
         return compiled_query
-        
 
     def get_organization_accounts(self) -> list:
         """ Retreive AWS Organization account """