Cleanup Spring Dependencies

[siri-varma]

Description

• I think having all the versions defined in the parent pom is a good idea.
• Moved the versions for testcontainer-module and spring-boot-example to parent pom. Depending on the feedback from this PR will work on rest of the modules in later prs.

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #940

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

[salaboy]

@siri-varma I've left some comments .. I do recommend not having the springframework.version managed in this way.. it will make it really hard to maintain.

[siri-varma]

@salaboy addressed all your comments. Thank you

[artur-ciocanu]

@siri-varma the PR looks good, but could you please provide some details why this is needed so I have a better high level understanding.

[siri-varma]

@artur-ciocanu / @salaboy / @dapr-java-sdk-reveiwers PR is ready to be reviewed. Thank you

[siri-varma]

@artur-ciocanu / @dapr-java-sdk-reviewer / @salaboy

Just a gentle reminder about this PR. Thank you

[artur-ciocanu]

@siri-varma could you address the conflicts and resend the PR. Thank you!

[artur-ciocanu]

@salaboy could you please take another look? From my perspective managing ALL the dependencies in a single place is a good idea. We used to have separate "code bases" for SDK and SDK tests, but almost a year ago I have added support for Maven profiles to be able to properly develop in an IDE.

So to recap I think moving all the deps to parent POM will allow us to upgrade more easily and have better control over dependencies.

[siri-varma]

@artur-ciocanu resolved the comments. Thank you

[artur-ciocanu]

LGTM, thanks for all the effort! 🙇

[artur-ciocanu]

@salaboy could you please take a look?

[salaboy]

@artur-ciocanu @siri-varma and @dapr/maintainers-java-sdk I think this is good to merge.. we need to try to define when to upgrade to 3.4.5 of spring boot. I need to check the pipelines to make sure that we are testing with that version.

[salaboy]

/LGTM

[salaboy]

@dapr/maintainers-java-sdk please merge

[siri-varma]

Thanks a lot everyone for the feedback and reviews

[siri-varma]

@cicoyle / @artur-ciocanu This PR is ready to be merged.

Created a separate issue to track the flaky build

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.17%. Comparing base (d759c53) to head (f52e021).
Report is 159 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #1334      +/-   ##
============================================
+ Coverage     76.91%   78.17%   +1.25%     
- Complexity     1592     1831     +239     
============================================
  Files           145      223      +78     
  Lines          4843     5663     +820     
  Branches        562      601      +39     
============================================
+ Hits           3725     4427     +702     
- Misses          821      916      +95     
- Partials        297      320      +23     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.