Skip to content

ARROW-2498: [Java] Use java 1.8 instead of java 1.7 - #1936

Closed
ghost wants to merge 2 commits into
apache:masterfrom
RMS:jdk8
Closed

ARROW-2498: [Java] Use java 1.8 instead of java 1.7#1936
ghost wants to merge 2 commits into
apache:masterfrom
RMS:jdk8

Conversation

@ghost

Copy link
Copy Markdown

No description provided.

@BryanCutler

Copy link
Copy Markdown
Member

Are we going to change Travis to just test Java 1.8 too?

@xhochy

Copy link
Copy Markdown
Member

@siddharthteotia@BryanCutler@jacques-n Is this ready to be merged?

@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.

+1

@BryanCutler

Copy link
Copy Markdown
Member

Also cc @icexelloss . I'll commit this later today if no more comments.

@BryanCutler

Copy link
Copy Markdown
Member

merged to master, thanks @agrove-rms !

Comment thread.travis.yml
- $TRAVIS_BUILD_DIR/ci/travis_install_linux.sh
script:
- $TRAVIS_BUILD_DIR/ci/travis_script_java.sh
- $TRAVIS_BUILD_DIR/ci/travis_script_javadoc.sh

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.

Why was this removed? Please revert and use JDK8

@wesm

wesm commented Apr 27, 2018

Copy link
Copy Markdown
Member

pribor pushed a commit to GlobalWebIndex/arrow that referenced this pull request Oct 24, 2025
Author: Andy Grove <andy.grove@rms.com>
Closesapache#1936 from agrove-rms/jdk8 and squashes the following commits:
d5dca81 <Andy Grove> remove jdk7 from CI matrix
ef01df4 <Andy Grove> use java 1.8 instead of java 1.7
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.

3 participants

@BryanCutler@xhochy@wesm