-
Notifications
You must be signed in to change notification settings - Fork 254
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.
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
update to latest azure spring boot release #64
Conversation
Issue related: microsoft/spring-data-cosmosdb#212 |
Please hold on this PR until Jenkins World demo finished by Step. 19th US time. BTW, I m not sure who is using this repo to do some demo, @Matthew-Dong Would u please send out a notification to the team? |
return new ResponseEntity<List<TodoItem>>(todoItemRepository.findAll(), HttpStatus.OK); | ||
final Iterable<TodoItem> itemIterable = todoItemRepository.findAll(); | ||
final List<TodoItem> items = new ArrayList<>(); | ||
itemIterable.forEach(todoItem -> items.add(todoItem)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just curious why not returning the Iterable directly?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is an update according to API change, previously findAll
returns List
, while new version returns Iterable
, just updated to remove the convert and the generics List<TodoItem>
type.
Seems the Jenkins World is closed and Spring Data updated to fix some issues, I'll close and merge it. |
No description provided.