Skip to content

Commit

Permalink
comments
Browse files Browse the repository at this point in the history
  • Loading branch information
emmyoop committed Jun 5, 2024
1 parent 8b1b0d6 commit 6a2dd30
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion dbt_common/record.py
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,7 @@ def __init__(self, current_recording_path: str, previous_recording_path: str) ->

def diff_query_records(self, current: List, previous: List) -> Dict[str, Any]:
# some of the table results are returned as a stringified list of dicts that don't diff because order isn't consistent
# convert it into a list of dicts so it can be diffed, ignoring order
for i in range(len(current)):
if current[i].get("result").get("table") is not None:
current[i] = json.loads(current[i]["result"]["table"])
Expand All @@ -89,7 +90,7 @@ def diff_default(self, current: List, previous: List) -> Dict[str, Any]:
return DeepDiff(current, previous, ignore_order=True, verbose_level=2)

def calculate_diff(self) -> Dict[str, Any]:
# TODO: should i convert the files to Records and diff the records?
# TODO: should i convert the files to Records and diff the records? blocked on this for now by MNTL-308

with open(self.current_recording_path) as current_recording:
current_dct = json.load(current_recording)
Expand Down

0 comments on commit 6a2dd30

Please sign in to comment.