odpi / egeria

Egeria core
https://egeria-project.org
Apache License 2.0
795 stars 259 forks source link

Monitor deprecation of RestTemplate #1087

Closed cmgrote closed 2 years ago

cmgrote commented 5 years ago

Monitor the eventual deprecation of RestTemplate from the Spring framework (see description in https://docs.spring.io/spring/docs/current/javadoc-api/index.html?org/springframework/web/client/RestTemplate.html) -- no specific release or timeline seems to be defined (yet), but the intent is there...

While we're keen to retain synchronous behaviour in the main clients to ease integration, it may be worth considering alternative clients that would also expose asynchronous capabilities (eg. to integrate with high-concurrency UIs or the like).

mandy-chessell commented 5 years ago

The Egeria code has a client connector that includes RestTemplate() - this is a place where the replacement calls and extended APIs can be added

cmgrote commented 4 years ago

Haven't seen any urgency that this is truly marked as deprecated, so suggest we push out beyond 1.4.

github-actions[bot] commented 4 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 20 days if no further activity occurs. Thank you for your contributions.

github-actions[bot] commented 4 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 20 days if no further activity occurs. Thank you for your contributions.

grahamwallis commented 4 years ago

The RestTemplate doesn't appear to have been deprecated and the comment in the javadoc that @cmgrote linked to talks about 'maintenance mode' rather than deprecation (with a view to future removal). I would suggest we let this close and (re-)open it if and when there we need to move from RestTemplate.

planetf1 commented 4 years ago

Given that all of our use should be captured in the spring rest connector (other than a few anomalies found in #3056 ) a change is likely fairly simple.

There's a noticeable difference though in the design (sync vs async at the detail level) and different dependencies.

Lots of interesting articles such as https://dzone.com/articles/resttemplate-vs-webclient

I wonder if it might be better to tackle this up-front in any case since we know it will go away

github-actions[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 20 days if no further activity occurs. Thank you for your contributions.

github-actions[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 20 days if no further activity occurs. Thank you for your contributions.

github-actions[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 20 days if no further activity occurs. Thank you for your contributions.

github-actions[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 20 days if no further activity occurs. Thank you for your contributions.

github-actions[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 20 days if no further activity occurs. Thank you for your contributions.

github-actions[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 20 days if no further activity occurs. Thank you for your contributions.