Skip to content
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

Using Supplier instead of Callable to get objects as lazy #771

Merged
merged 1 commit into from
Sep 1, 2014

Conversation

garcia-jj
Copy link
Member

Because Supplier is the right type to get objects using lazy approach.

Since we are using Java 7 without lambdas, we can use Guava Supplier. And when we decide to use Java 8, will too easy to migrate.

Cache is a internal feature, designed to use only in our internal code. So there is no problems to change methods signature.

@lucascs
Copy link
Member

lucascs commented Sep 1, 2014

👍 but worth a note on the changelog

@garcia-jj
Copy link
Member Author

Can we add notes in changelog before release?

@lucascs
Copy link
Member

lucascs commented Sep 1, 2014

If I'm not wrong we can draft the next release:
https://github.com/caelum/vraptor4/releases

@Turini
Copy link
Member

Turini commented Sep 1, 2014

if you want to, just add the note at releasing.txt file :)

garcia-jj added a commit that referenced this pull request Sep 1, 2014
Using Supplier instead of Callable to get objects as lazy
@garcia-jj garcia-jj merged commit 9f10115 into master Sep 1, 2014
@garcia-jj garcia-jj deleted the ot-supplierinsteadofcallable branch September 1, 2014 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants