Skip to content

Support New Relic on Java8#78

Merged
nebhale merged 1 commit intocloudfoundry:masterfrom
cgfrost:master
Aug 6, 2014
Merged

Support New Relic on Java8#78
nebhale merged 1 commit intocloudfoundry:masterfrom
cgfrost:master

Conversation

@cgfrost
Copy link
Contributor

@cgfrost cgfrost commented Aug 6, 2014

New Relic has experimental support for Java 8 that isn't included in the
buildpack. This change adds support for New Relic on Java 8 by adding the
appropriate flag to JAVA_OPTS when Java 8 is selected and the application is
bound to a New Relic service.

See: https://discuss.newrelic.com/t/a-note-on-java-8-support/1199/2

[#75561442]

@cfdreddbot
Copy link

Hey cgfrost!

Thanks for submitting this pull request!

All pull request authors must have a Contributor License Agreement (CLA) on-file with us. Please sign the appropriate CLA (individual or corporate).

When sending signed CLA please provide your github username in case of individual CLA or the list of github usernames that can make pull requests on behalf of your organization.

Copy link
Contributor

Choose a reason for hiding this comment

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

Needs documentation See attr_accessor :root.

New Relic has experimental support for Java 8 that isn't included in the
buildpack. This change adds support for New Relic on Java 8 by adding the
appropriate flag to JAVA_OPTS when Java 8 is selected and the application is
bound to a New Relic service.

See: https://discuss.newrelic.com/t/a-note-on-java-8-support/1199/2

[#75561442]
@cgfrost
Copy link
Contributor Author

cgfrost commented Aug 6, 2014

Add comment

@nebhale nebhale merged commit 59098a9 into cloudfoundry:master Aug 6, 2014
nebhale added a commit that referenced this pull request Aug 6, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants