spruce up CLI output in dbt context

[leoebfolsom]

My attempt at improving the CLI printout without using any additional python packages.

BEFORE:

AFTER:

TODO based on comments in dlawin#3:

• @erezsh: Looks like _get_stats_dbt() should instead just be a flag to _get_stats() lightweight cli output update dlawin/data-diff#3 (comment)
• @erezsh: Better to implement this using zip() / safezip() lightweight cli output update dlawin/data-diff#3 (comment)
• @dlawin: Since we're adding a parameter here, we'll need to account for it in the other DiffStats() in this file lightweight cli output update dlawin/data-diff#3 (comment)
• @dlawin : update get_stats_string(self, is_dbt:bool = False): ... add the additional functionality there and just wrap it in an if is_dbt: lightweight cli output update dlawin/data-diff#3 (comment)
• @dlawin : line length

[leoebfolsom]

@erezsh first, please note that this is a new PR now that Dan's dbt integration is merged into master.

I've made all the changes suggested by you and Dan in the previous PR except for using zip() / unzip(). I found some explainers since zip() is new to me, but I'm not sure how they would apply here.

My goal is to add to keep track of conflicts for each of the extra_columns as I iterate through the rows of result_list, and store the counts in a dictionary: { first_name: 14, last_name: 0, street: 0, city: 2, state: 0, ... } representing the # of conflicts per column.

I can see how it's repetitive to iterate through for i in range(0,len(extra_columns)): for each row and then check the value based on the index (if extra_column_values[i] != extra_column_values_store[k][i]:) for every column.

However, I'm struggling to see where I could gain efficiency using zip(). Can you please be more specific about what you have in mind and/or provide an example?

[dlawin]

Approving with the performance/readability caveats I've mentioned