all snapshot_meta_column_names including dbt_is_deleted should be a string input not a boolean

[graciegoheen]

Contributions

• I have read the contribution docs, and understand what's expected of me.

Link to the page on docs.getdbt.com requiring updates

https://docs.getdbt.com/reference/resource-configs/snapshot_meta_column_names

What part(s) of the page would you like to see updated?

Right now it says to input a boolean for snapshot_meta_column_names dbt_is_deleted:

snapshots:
  - name: <snapshot_name>
    config:
      snapshot_meta_column_names:
        dbt_valid_from: <string>
        dbt_valid_to: <string>
        dbt_scd_id: <string>
        dbt_updated_at: <string>
        dbt_is_deleted: <boolean>

Instead it should be a string:

snapshots:
  - name: <snapshot_name>
    config:
      snapshot_meta_column_names:
        dbt_valid_from: <string>
        dbt_valid_to: <string>
        dbt_scd_id: <string>
        dbt_updated_at: <string>
        dbt_is_deleted: <string>

Additional information

No response

[mirnawong1]

hey @graciegoheen , so since it's a string - the user can still input 'yes' or 'no'? or 'true' and 'false'? or is_deleted?

currently the definition for dbt_deleted_at is: A boolean value indicating if the record has been deleted. True if deleted, False otherwise.

[graciegoheen]

That's the correct definition for the actual metadata field dbt_is_deleted but for this snapshot_meta_column_names config, you just map the dbt-provided name with what you want the name to be. So maybe you want the dbt_is_deleted field to be named grace_delete_column. The thing you input for snapshot_meta_column_names: {dbt_is_deleted: grace_delete_column} is a string. Does that make sense?

[mirnawong1]

makes sense, thank you! i'll change the description because it still references the boolean (true/false). thank you!