Skip to content

Commit 1eec42b

Browse files
Iakov Gandarken99
authored andcommitted
better messages for update
1 parent 54e86d6 commit 1eec42b

File tree

1 file changed

+7
-4
lines changed

1 file changed

+7
-4
lines changed

cid/common.py

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -863,7 +863,7 @@ def create_or_update_dataset(self, dataset_definition: dict, arn: str=None,recur
863863
# Read dataset definition from template
864864
dataset_file = dataset_definition.get('File')
865865
if not dataset_file:
866-
logger.critical(f"Error: {dataset_definition.get('Name')} definition is broken")
866+
logger.critical("Error: definition is broken. Check resources file.")
867867
exit(1)
868868

869869
if not len(self.qs.athena_datasources):
@@ -878,13 +878,16 @@ def create_or_update_dataset(self, dataset_definition: dict, arn: str=None,recur
878878
resource_name=f'data/datasets/{dataset_file}',
879879
).decode('utf-8'))
880880

881+
pre_compiled_dataset = json.loads(template.safe_substitute())
882+
dataset_name = pre_compiled_dataset.get('Name')
883+
881884
# let's find the schema/database name
882885
schemas = []
883886
if arn:
884887
dataset_id = arn.split('/')[-1]
885888
schemas = list(set(self.qs.get_datasets(id=dataset_id)[0].schemas))
886889
else: # try to find dataset and get athena database
887-
found_datasets = self.qs.get_datasets(name=dataset_definition.get('Name'))
890+
found_datasets = self.qs.get_datasets(name=dataset_name)
888891
if found_datasets:
889892
schemas = list(set(sum([d.schemas for d in found_datasets], [])))
890893

@@ -929,8 +932,8 @@ def create_or_update_dataset(self, dataset_definition: dict, arn: str=None,recur
929932
if isinstance(found_dataset, Dataset):
930933
if update:
931934
self.qs.update_dataset(compiled_dataset)
932-
elif found_dataset.name != dataset_definition.get('Name'):
933-
print(f'Dataset found with name {found_dataset.name}, but {dataset_definition.get('Name')} expected. Updating.')
935+
elif found_dataset.name != dataset_name:
936+
print(f"Dataset found with name {found_dataset.name}, but {dataset_name} expected. Updating.")
934937
self.qs.update_dataset(compiled_dataset)
935938
else:
936939
print(f'No update requested for dataset {compiled_dataset.get("DataSetId")}')

0 commit comments

Comments
 (0)