Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Apr 23, 2018

No description provided.

@BryanCutler
Copy link
Member

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

@xhochy
Copy link
Member

xhochy commented Apr 25, 2018

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

Copy link
Member

@BryanCutler BryanCutler left a comment

Choose a reason for hiding this comment

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

+1

@BryanCutler
Copy link
Member

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

@BryanCutler
Copy link
Member

merged to master, thanks @agrove-rms !

- $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
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
Copy link
Member

wesm commented Apr 27, 2018

pribor pushed a commit to GlobalWebIndex/arrow that referenced this pull request Oct 24, 2025
Author: Andy Grove <andy.grove@rms.com>

Closes apache#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 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