Adds Global TTL Configuration for DynamoDB State Store

[ZeynelKoca]

Description

Adds a way to globally configure the state TTL for dynamo (inspired by #1059).

The defined default TTL will only be used if the state request does not explicitly specify a TTL itself.

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

• Code compiles correctly
• Created/updated tests
• Extended the documentation
  • Created the dapr/docs PR: Describe new ttlInSeconds attribute in DynamoDB state store setup docs#4911

[JoostPV]

Would be nice when the different statestores (Redis and Dynamo) behave in a similar way.

[MyMirelHub]

There will be a slight delay in reviewing the DynamoDB-related PRs. We need to address the missing/disabled tests first, which are being tracked in issue #4047. Thanks for your patience!

[ZeynelKoca]

@sicoyle Ready for another review round

[sicoyle]

one last comment and then this LGTM - thank you!

[ZeynelKoca]

one last comment and then this LGTM - thank you!

Fixed. Also updated the docs PR with the same description

[sicoyle]

@ZeynelKoca pls see the build failures and ping when you're ready and I can retrigger 🙏

[ZeynelKoca]

@sicoyle Fixed a stupid rebase mistake. Try again?

[ZeynelKoca]

@sicoyle Can the pipeline run get an approval?

[ZeynelKoca]

@sicoyle Fixed the formatting 🙏

[ZeynelKoca]

@sicoyle I think a re-run is needed? Pipelines failed due to being unable to checkout the repo

[sicoyle]

@sicoyle I think a re-run is needed? Pipelines failed due to being unable to checkout the repo

rerunning, lets see. I know github has been having quite a few issues recently

[ZeynelKoca]

@sicoyle I think a re-run is needed? Pipelines failed due to being unable to checkout the repo

rerunning, lets see. I know github has been having quite a few issues recently

Fixed another formatting issue 🙏

[ZeynelKoca]

@sicoyle Can you trigger a pipeline run? I see that the corresponding docs PR has already been merged.

[sicoyle]

as long as the build passes, then LGTM. Thank you for your contribution and iterating on feedback 🤗

[ZeynelKoca]

@sicoyle Thanks! The build failed again due to formatting issues (🙄) so I upgraded to the exact go version the pipeline was using. go fmt now properly added the missing new-line at the end of the dynamodb_test.go file. Another pipeline run should do it 🤞

[sicoyle]

The build failed again due to formatting issues (🙄)

LOL I feel this 😄 It's running. Will circle back later today and if all green we can send it 🚀

[ZeynelKoca]

@sicoyle No formatting issues anymore 🎉

1 failing unit test though, which I updated based on the new TTL property behavior. All tests pass now locally

[ZeynelKoca]

@sicoyle No formatting issues anymore 🎉

1 failing unit test though, which I updated based on the new TTL property behavior. All tests pass now locally

@sicoyle Bump

[github-actions]

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 7 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

[ZeynelKoca]

@cicoyle Any plans on merging this PR? The docs PR was merged a while ago and is already live in production while the actual functionality (this PR) isn't.