Details

    • Hadoop Flags:
      Incompatible change, Reviewed
    • Release Note:
      ZooKeeper now requires Java 6.

      Description

      We should require Java 6 for release 3.1.0. At this point there is full java6 support on all major platforms (linux/windows/macosx).
      The vast majority of ZooKeeper contributors use Java 6 when developing, and hence do run into (usually minor) back-compatibility problems.
      I believe there are also some benefits regarding JMX (updated spec support in java6) if we drop java 5.

      FWIW: Hadoop Core has also moved to require Java6:
      https://issues.apache.org/jira/browse/HADOOP-2325

        Activity

        Hide
        mahadev Mahadev konar added a comment -

        the main reason we moved to java 6 in hadoop was performance with NIO in java 6. +1 for moving to java 6.

        Show
        mahadev Mahadev konar added a comment - the main reason we moved to java 6 in hadoop was performance with NIO in java 6. +1 for moving to java 6.
        Hide
        phunt Patrick Hunt added a comment -

        Update to java6:

        1) updated docs to spec 1.6 as min
        2) updated build.xml to use java6 api (link to api docs on java.sun)
        3) updated zkServer.sh and docs to use Managed*Main versions (jmx) of server.

        I didn't explicitly require 1.6 in the build.xml, don't think that's necessary at least for the time being.

        I will update the JMX specific documentation as part of ZOOKEEPER-177

        Show
        phunt Patrick Hunt added a comment - Update to java6: 1) updated docs to spec 1.6 as min 2) updated build.xml to use java6 api (link to api docs on java.sun) 3) updated zkServer.sh and docs to use Managed*Main versions (jmx) of server. I didn't explicitly require 1.6 in the build.xml, don't think that's necessary at least for the time being. I will update the JMX specific documentation as part of ZOOKEEPER-177
        Hide
        phunt Patrick Hunt added a comment -

        submitting patch and marking incompatible wrt the required java version is bumped up to 6 – we need to document this in the release notes and changes file as incompatible.

        Show
        phunt Patrick Hunt added a comment - submitting patch and marking incompatible wrt the required java version is bumped up to 6 – we need to document this in the release notes and changes file as incompatible.
        Hide
        breed Benjamin Reed added a comment -

        Committed revision 736673.

        Show
        breed Benjamin Reed added a comment - Committed revision 736673.
        Hide
        hudson Hudson added a comment -

        Integrated in ZooKeeper-trunk #207 (See http://hudson.zones.apache.org/hudson/job/ZooKeeper-trunk/207/)
        , Require Java 6.

        Show
        hudson Hudson added a comment - Integrated in ZooKeeper-trunk #207 (See http://hudson.zones.apache.org/hudson/job/ZooKeeper-trunk/207/ ) , Require Java 6.

          People

          • Assignee:
            phunt Patrick Hunt
            Reporter:
            phunt Patrick Hunt
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development