Skip to content
This repository has been archived by the owner on Oct 5, 2021. It is now read-only.

ISPN-3039: Adding a quickstart for JCache #21

Closed
wants to merge 1 commit into from
Closed

ISPN-3039: Adding a quickstart for JCache #21

wants to merge 1 commit into from

Conversation

rpelisse
Copy link
Contributor

First stab at a quickstart for JCache - probably needs some enhancement, but at least, it's a start !

resources, i.e. build is platform dependent! -->
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>

<infinispan.version>7.0.0.Final</infinispan.version>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You might want to upgrade to 7.1.0.Final

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, I started with 7.1.0.Final but went back to 7.0.0.Final to align with the other quickstart:

https://github.com/infinispan/infinispan-quickstart/blob/master/embedded-cache/pom.xml#L49

If you want, I can try to update all the quickstart to 7.1.0.Final ? Actually, maybe the ISPN version should lives in the parent of the quickstart, so that updating requires only one file change ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, just googled this out, and there is no way apparently to have the ISPN version in the parent pom of all the quickstart.

So I'll work on a separate PR updating all Quickstart to 7.1.0.Final.

@galderz
Copy link
Member

galderz commented Feb 12, 2015

Looks great! If anything, I'd add some further examples of the API, showing for example the difference between put and getAndPut. IOW, the quickstart could add some focus on those APIs that are different to what Infinispan/java.util.Map/java.util.concurrent.ConcurrentMap users are used to. The documentation contains some tables on the differences which can guide you on adding some more examples.

@rpelisse
Copy link
Contributor Author

Superseed by Pull Request 22

@rpelisse rpelisse closed this Feb 12, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants