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

Archaius 0.x is not compatible with Java 6/7 #522

Merged
merged 1 commit into from
Nov 8, 2017

Conversation

rkettelerij
Copy link

@rkettelerij rkettelerij commented Oct 17, 2017

The following issue is currently causing issues in the spring-cloud project, since spring-cloud is currently based on spring 4.x which maintains compatibility with Java 7.

Archaius 0.x claims to be compatible with Java 6 (source and target versions are set to 1.6) but when calling com.netflix.config.ConcurrentMapConfiguration.getKeys under Java 7 the following exception occurs:

Caused by: java.lang.NoSuchMethodError: java.util.concurrent.ConcurrentHashMap.keySet()Ljava/util/concurrent/ConcurrentHashMap$KeySetView; at com.netflix.config.ConcurrentMapConfiguration.getKeys(ConcurrentMapConfiguration.java:152)

This is caused by the fact that Archaius 0.x is compiled using a JDK 8 with source/target set to 1.6 but without a bootclasspath set to 1.6. Therefore you'll run into this issue: https://gist.github.com/AlainODea/1375759b8720a3f9f094

I see three solutions:

@rkettelerij rkettelerij changed the title Fix Java 7 <> Java 8 compatibility issue Archaius 1.x is not compatible with Java 6/7 Oct 17, 2017
@rkettelerij rkettelerij changed the title Archaius 1.x is not compatible with Java 6/7 Archaius 0.x is not compatible with Java 6/7 Oct 17, 2017
spencergibb added a commit to spring-cloud/spring-cloud-netflix that referenced this pull request Oct 17, 2017
pway99 pushed a commit to pway99/spring-cloud-netflix that referenced this pull request Oct 24, 2017
@twicksell twicksell merged commit 8fa8499 into Netflix:master Nov 8, 2017
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.

2 participants