Skip to content

ARROW-2518: [Java] Re-instate JDK tests in matrix, but with JDK 8 instead of JDK 7 - #1956

Closed
ghost wants to merge 1 commit into
apache:masterfrom
RMS:restore_java_tests
Closed

ARROW-2518: [Java] Re-instate JDK tests in matrix, but with JDK 8 instead of JDK 7#1956
ghost wants to merge 1 commit into
apache:masterfrom
RMS:restore_java_tests

Conversation

@ghost

Copy link
Copy Markdown

No description provided.

@ghostghost changed the title re-instate JDK tests in matrix, but with JDK 8 instead of JDK 7ARROW-2518: [Java] Re-instate JDK tests in matrix, but with JDK 8 instead of JDK 7Apr 27, 2018
@BryanCutler

Copy link
Copy Markdown
Member

Sorry, my fault. I saw there was OpenJDK8 already there, but missed that it was for integration tests. I should have looked closer.

@BryanCutlerBryanCutler left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM based on a diff. Not sure why Travis is failing, is it unrelated?

@paddyhoran

Copy link
Copy Markdown
Contributor

@BryanCutler I think it's unrelated see ARROW-2509. The same thing is happening on 1949

@BryanCutler

Copy link
Copy Markdown
Member

Great, that's for pointing out the JIRA @paddyhoran , I'll go ahead and commit this since it passed OpenJDK8 tests.

Sign up for freeto 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

@BryanCutler@paddyhoran