Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.8.0
    • Fix Version/s: 1.8.0
    • Component/s: None
    • Labels:
      None

      Description

      We should move to Java 8 as a minimum requirement.

      1. Java 7 is EOL'ed http://www.oracle.com/technetwork/java/eol-135779.html.
      2. Many projects are adopting java 8 as a minimum requirement, for instance:

      The attached patch was tested on Ubuntu 16.04 with openjdk version "1.8.0_91".
      All unit tests, except for the known flaky ones, successfully passed.

        Issue Links

          Activity

          Hide
          denes Denes Arvay added a comment -

          Thanks for the patch Lior Zeno, thank you for the review Attila Simon

          Show
          denes Denes Arvay added a comment - Thanks for the patch Lior Zeno , thank you for the review Attila Simon
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Jenkins build Flume-trunk-hbase-1 #250 (See https://builds.apache.org/job/Flume-trunk-hbase-1/250/)
          FLUME-2945. Bump java target version to 1.8 (denes: http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=33d19046f0db66c3093f68716e4f6100301d3b49)

          • (edit) flume-ng-doc/sphinx/FlumeUserGuide.rst
          • (edit) README.md
          • (edit) DEVNOTES
          • (edit) conf/flume-env.sh.template
          • (edit) flume-ng-sources/flume-taildir-source/pom.xml
          • (edit) pom.xml
          • (edit) .travis.yml
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Jenkins build Flume-trunk-hbase-1 #250 (See https://builds.apache.org/job/Flume-trunk-hbase-1/250/ ) FLUME-2945 . Bump java target version to 1.8 (denes: http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=33d19046f0db66c3093f68716e4f6100301d3b49 ) (edit) flume-ng-doc/sphinx/FlumeUserGuide.rst (edit) README.md (edit) DEVNOTES (edit) conf/flume-env.sh.template (edit) flume-ng-sources/flume-taildir-source/pom.xml (edit) pom.xml (edit) .travis.yml
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 33d19046f0db66c3093f68716e4f6100301d3b49 in flume's branch refs/heads/trunk from Lior Zeno
          [ https://git-wip-us.apache.org/repos/asf?p=flume.git;h=33d1904 ]

          FLUME-2945. Bump java target version to 1.8

          • Removed the unsupported PermSize and MaxPermSize settings from .travis.yml
          • Updated DEVNOTES, README and Flume User Guide
          • Removed the maven-compiler-plugin from the taildir-source subproject
          • Changed the sourceJavaVersion and targetJavaVersion to 1.8 in the root pom.xml

          (Lior Zeno via Denes Arvay)

          Show
          jira-bot ASF subversion and git services added a comment - Commit 33d19046f0db66c3093f68716e4f6100301d3b49 in flume's branch refs/heads/trunk from Lior Zeno [ https://git-wip-us.apache.org/repos/asf?p=flume.git;h=33d1904 ] FLUME-2945 . Bump java target version to 1.8 Removed the unsupported PermSize and MaxPermSize settings from .travis.yml Updated DEVNOTES, README and Flume User Guide Removed the maven-compiler-plugin from the taildir-source subproject Changed the sourceJavaVersion and targetJavaVersion to 1.8 in the root pom.xml (Lior Zeno via Denes Arvay)
          Hide
          denes Denes Arvay added a comment -

          Hi Lior Zeno, I think this patch is ready to go. I'm going to do a final review and if nobody has any concerns I'll commit it.

          Show
          denes Denes Arvay added a comment - Hi Lior Zeno , I think this patch is ready to go. I'm going to do a final review and if nobody has any concerns I'll commit it.
          Hide
          mpercy Mike Percy added a comment -

          I wouldn't be opposed to this after we're done with the Flume 1.7.0 release

          Show
          mpercy Mike Percy added a comment - I wouldn't be opposed to this after we're done with the Flume 1.7.0 release

            People

            • Assignee:
              liorze Lior Zeno
              Reporter:
              liorze Lior Zeno
            • Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development