Add DynamoDB table detect and correct pipeline #34

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

misraved merged 11 commits into release/v0.2.0 from dynamdb-stale-data

May 17, 2024

Contributor

khushboo9024 commented May 15, 2024

No description provided.


          Add DynamoDB table delete pipeline

06e10b8

khushboo9024 self-assigned this

graza-io reviewed

View reviewed changes

dynamodb/dynamodb_table_stale_data.fp Outdated Show resolved Hide resolved

graza-io reviewed

View reviewed changes

dynamodb/dynamodb_table_stale_data.fp Outdated Show resolved Hide resolved

graza-io reviewed

View reviewed changes

dynamodb/dynamodb_table_stale_data.fp Outdated Show resolved Hide resolved

graza-io reviewed

View reviewed changes

dynamodb/dynamodb_table_stale_data.fp Outdated Show resolved Hide resolved

graza-io reviewed

View reviewed changes

thrifty.fpvars.example Outdated Show resolved Hide resolved

graza-io requested changes

View reviewed changes

Collaborator

graza-io left a comment

See comments.

Additionally @misraved - I feel like the wording is wrong here: "stale data exceeding max age"... perhaps reads better as "with stale data", thoughts?

Contributor

misraved commented May 17, 2024

@graza-io agreed, I think with_stale_data is self-explanatory and we should go with it 👍.

khushboo9024 and others added 10 commits

May 17, 2024 14:35


          update

3463fc0


          update

6ffa576


          Update mod.fp

5b1e33c


          update

89a684e


          update

39375bf


          Update thrifty.fpvars.example

275efee


          Fix typos

b2a3c9d


          Sort local variables

7a327e0


          fix: fixed pluralisation

1ed15d7


          fix: renamed variable

6cc40e4

graza-io approved these changes

View reviewed changes

misraved approved these changes

View reviewed changes

misraved changed the title ~~Add DynamoDB table delete pipeline~~ Add DynamoDB table detect and correct pipeline

misraved merged commit d93fcce into release/v0.2.0

misraved deleted the dynamdb-stale-data branch

May 17, 2024 14:16

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet