Update all CDAP apps/guides/archetypes have an explicit dependency on guava if they need it

Description

After guava dependency got removed from cdap-api all apps/guides/archetype which uses guava needs to have it as their dependency.
We will need to go through all of them and make verify this.

Release Notes

None

Activity

Show:

Rohit SinhaApril 29, 2016 at 6:53 AM
Edited

Looks like you did updated the guides to use guava but missed few which is totally because them being in their independent repo. So hard to track and manage. I have updated the remaining ones. All good.

Rohit SinhaApril 29, 2016 at 6:14 AM
Edited

In the above comment

All the guides have been updated. PR for cdap-apps: https://github.com/caskdata/cdap-apps/pull/110.

Takes to cdap-apps repo. I don't see any PR in which guava was added to guides and the mvn build fails too.

Can you confirm whether you added guava to guides ?
https://github.com/cdap-guides

Bhooshan MogalApril 19, 2016 at 11:53 PM

Updated the cdap-apps repo as well as all the cdap-guides repos. Verified that the cdap archetypes do not call out guava as provided anymore. Going to close this issue. Please re-open if any more repos need to be updated.

Bhooshan MogalApril 19, 2016 at 3:17 PM

All the guides have been updated. PR for cdap-apps: https://github.com/caskdata/cdap-apps/pull/110.

: Are there any other repos that need to be verified?

Bhooshan MogalApril 18, 2016 at 11:02 PM

Fixed
Pinned fields
Click on the next to a field label to start pinning.

Details

Assignee

Reporter

Affects versions

Components

Fix versions

Priority

Created March 15, 2016 at 9:47 PM
Updated April 29, 2016 at 6:54 AM
Resolved April 19, 2016 at 11:53 PM