snowplow / snowplow-rdb-loader

Stores Snowplow enriched events in Redshift, Snowflake and Databricks
Other
31 stars 17 forks source link

Rename disableMigration to disableRecovery #1337

Closed istreeter closed 10 months ago

istreeter commented 10 months ago

As discussed elsewhere, disableRecovery is a better name for the feature flag... because that is what it does. When set, we do not load data into a recovery table.

As a side effect, we also don't migrate the table. But that's not the main purpose of the feature flag.

I did not create a github issue for this, because I think we can squash it into the original feature commit before release. Shout though if you think it will be easier to manage the branch if this has an issue to link to.