Skip to content

SAMOA-76: changes JDK used for Travis builds #74

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

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

mgrzenda
Copy link
Contributor

SAMOA-76: changes JDK used for Travis builds

@mgrzenda
Copy link
Contributor Author

Here is a brief summary of findings and solution:

  1. oraclejdk7 is no longer available in Travis. Hence, the old Travis configuration failed.
  2. openjdk7 was a suggested replacement
  3. However, Travis builds based on openjdk7 are started, but never finish (are killed because of inactivity)
  4. This kind of hanging build can be obtained also on standalone Linux/ mvn if _JAVA_OPTIONS are set as in Travis i.e. to "-Xmx2048m -Xms512m" and openjdk7 is used
  5. Increasing memory limits in standalone mvn run resolves the problem 4), but not in Travis builds. Travis builds are still hanging in HDFS-related tests, as before even with less restrictive memory settings and forced garbage collection (see the list of commits to this JIRA issue for different options)
  6. The ultimate working solution is to replace oraclejdk7 with oraclejdk8. This seems to be acceptable solution.

@mgrzenda
Copy link
Contributor Author

mgrzenda commented Oct 5, 2018

Shall we apply this change of JDK7 to JDK8 to fix Travis builds?

@pwawrzyniak
Copy link
Contributor

+1

@csterling
Copy link
Contributor

It looks like this change is no longer sufficient. Newer builds are failing with an error indicating that Java feature-set 9 is the minimum requirement now. There may be other errors with the build after that is resolved. Requires further investigation.

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