Uploaded image for project: 'Hadoop Common'
  1. Hadoop Common
  2. HADOOP-11356

Removed deprecated o.a.h.fs.permission.AccessControlException

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.0.0-alpha1
    • Component/s: None
    • Labels:
      None
    • Hadoop Flags:
      Incompatible change, Reviewed
    • Release Note:
      org.apache.hadoop.fs.permission.AccessControlException was deprecated in the last major release, and has been removed in favor of org.apache.hadoop.security.AccessControlException

      Description

      The o.a.h.fs.permission.AccessControlException has been deprecated for last major releases and it should be removed.

        Issue Links

          Activity

          Hide
          wheat9 Haohui Mai added a comment -

          Reverted in branch-2.

          Show
          wheat9 Haohui Mai added a comment - Reverted in branch-2.
          Hide
          stevel@apache.org Steve Loughran added a comment -

          HIVE-9115 is the patch there; if we revert the patch in branch-2 it'll keep existing hive releases happy

          Show
          stevel@apache.org Steve Loughran added a comment - HIVE-9115 is the patch there; if we revert the patch in branch-2 it'll keep existing hive releases happy
          Hide
          wheat9 Haohui Mai added a comment -

          Sounds good to me. Since it has been deprecated in branch-1, maybe we need to put a fix in Hive as well.

          Show
          wheat9 Haohui Mai added a comment - Sounds good to me. Since it has been deprecated in branch-1, maybe we need to put a fix in Hive as well.
          Hide
          stevel@apache.org Steve Loughran added a comment -

          This has unfortunately broken hive building on 2.7; and may also stop it from working.

          I propose reverting this in branch-2 & noting as an incompatible in trunk

          Show
          stevel@apache.org Steve Loughran added a comment - This has unfortunately broken hive building on 2.7; and may also stop it from working. I propose reverting this in branch-2 & noting as an incompatible in trunk
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Mapreduce-trunk-Java8 #27 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/27/)
          HADOOP-11356. Removed deprecated o.a.h.fs.permission.AccessControlException. Contributed by Li Lu. (wheat9: rev 2829b7a96ffe6d2ca5e81689c7957e4e97042f2d)

          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/AccessControlException.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/AccessControlException.java
          • hadoop-common-project/hadoop-common/CHANGES.txt
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Mapreduce-trunk-Java8 #27 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/27/ ) HADOOP-11356 . Removed deprecated o.a.h.fs.permission.AccessControlException. Contributed by Li Lu. (wheat9: rev 2829b7a96ffe6d2ca5e81689c7957e4e97042f2d) hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/AccessControlException.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/AccessControlException.java hadoop-common-project/hadoop-common/CHANGES.txt
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Mapreduce-trunk #1981 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1981/)
          HADOOP-11356. Removed deprecated o.a.h.fs.permission.AccessControlException. Contributed by Li Lu. (wheat9: rev 2829b7a96ffe6d2ca5e81689c7957e4e97042f2d)

          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/AccessControlException.java
          • hadoop-common-project/hadoop-common/CHANGES.txt
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/AccessControlException.java
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Mapreduce-trunk #1981 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1981/ ) HADOOP-11356 . Removed deprecated o.a.h.fs.permission.AccessControlException. Contributed by Li Lu. (wheat9: rev 2829b7a96ffe6d2ca5e81689c7957e4e97042f2d) hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/AccessControlException.java hadoop-common-project/hadoop-common/CHANGES.txt hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/AccessControlException.java
          Hide
          hudson Hudson added a comment -

          SUCCESS: Integrated in Hadoop-Hdfs-trunk #1959 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1959/)
          HADOOP-11356. Removed deprecated o.a.h.fs.permission.AccessControlException. Contributed by Li Lu. (wheat9: rev 2829b7a96ffe6d2ca5e81689c7957e4e97042f2d)

          • hadoop-common-project/hadoop-common/CHANGES.txt
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/AccessControlException.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/AccessControlException.java
          Show
          hudson Hudson added a comment - SUCCESS: Integrated in Hadoop-Hdfs-trunk #1959 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1959/ ) HADOOP-11356 . Removed deprecated o.a.h.fs.permission.AccessControlException. Contributed by Li Lu. (wheat9: rev 2829b7a96ffe6d2ca5e81689c7957e4e97042f2d) hadoop-common-project/hadoop-common/CHANGES.txt hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/AccessControlException.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/AccessControlException.java
          Hide
          hudson Hudson added a comment -

          SUCCESS: Integrated in Hadoop-Hdfs-trunk-Java8 #27 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/27/)
          HADOOP-11356. Removed deprecated o.a.h.fs.permission.AccessControlException. Contributed by Li Lu. (wheat9: rev 2829b7a96ffe6d2ca5e81689c7957e4e97042f2d)

          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/AccessControlException.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/AccessControlException.java
          • hadoop-common-project/hadoop-common/CHANGES.txt
          Show
          hudson Hudson added a comment - SUCCESS: Integrated in Hadoop-Hdfs-trunk-Java8 #27 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/27/ ) HADOOP-11356 . Removed deprecated o.a.h.fs.permission.AccessControlException. Contributed by Li Lu. (wheat9: rev 2829b7a96ffe6d2ca5e81689c7957e4e97042f2d) hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/AccessControlException.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/AccessControlException.java hadoop-common-project/hadoop-common/CHANGES.txt
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Yarn-trunk #766 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/766/)
          HADOOP-11356. Removed deprecated o.a.h.fs.permission.AccessControlException. Contributed by Li Lu. (wheat9: rev 2829b7a96ffe6d2ca5e81689c7957e4e97042f2d)

          • hadoop-common-project/hadoop-common/CHANGES.txt
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/AccessControlException.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/AccessControlException.java
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Yarn-trunk #766 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/766/ ) HADOOP-11356 . Removed deprecated o.a.h.fs.permission.AccessControlException. Contributed by Li Lu. (wheat9: rev 2829b7a96ffe6d2ca5e81689c7957e4e97042f2d) hadoop-common-project/hadoop-common/CHANGES.txt hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/AccessControlException.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/AccessControlException.java
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Yarn-trunk-Java8 #27 (See https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/27/)
          HADOOP-11356. Removed deprecated o.a.h.fs.permission.AccessControlException. Contributed by Li Lu. (wheat9: rev 2829b7a96ffe6d2ca5e81689c7957e4e97042f2d)

          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/AccessControlException.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/AccessControlException.java
          • hadoop-common-project/hadoop-common/CHANGES.txt
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Yarn-trunk-Java8 #27 (See https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/27/ ) HADOOP-11356 . Removed deprecated o.a.h.fs.permission.AccessControlException. Contributed by Li Lu. (wheat9: rev 2829b7a96ffe6d2ca5e81689c7957e4e97042f2d) hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/AccessControlException.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/AccessControlException.java hadoop-common-project/hadoop-common/CHANGES.txt
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-trunk-Commit #6653 (See https://builds.apache.org/job/Hadoop-trunk-Commit/6653/)
          HADOOP-11356. Removed deprecated o.a.h.fs.permission.AccessControlException. Contributed by Li Lu. (wheat9: rev 2829b7a96ffe6d2ca5e81689c7957e4e97042f2d)

          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/AccessControlException.java
          • hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/AccessControlException.java
          • hadoop-common-project/hadoop-common/CHANGES.txt
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-trunk-Commit #6653 (See https://builds.apache.org/job/Hadoop-trunk-Commit/6653/ ) HADOOP-11356 . Removed deprecated o.a.h.fs.permission.AccessControlException. Contributed by Li Lu. (wheat9: rev 2829b7a96ffe6d2ca5e81689c7957e4e97042f2d) hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/AccessControlException.java hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/AccessControlException.java hadoop-common-project/hadoop-common/CHANGES.txt
          Hide
          wheat9 Haohui Mai added a comment -

          I've committed the patch to trunk and branch-2. Thanks Li Lu for the contribution.

          Show
          wheat9 Haohui Mai added a comment - I've committed the patch to trunk and branch-2. Thanks Li Lu for the contribution.
          Hide
          wheat9 Haohui Mai added a comment -

          +1. I'll commit it shortly.

          Show
          wheat9 Haohui Mai added a comment - +1. I'll commit it shortly.
          Hide
          hadoopqa Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12685254/HDFS-7479-120414.patch
          against trunk revision 0653918.

          +1 @author. The patch does not contain any @author tags.

          -1 tests included. The patch doesn't appear to include any new or modified tests.
          Please justify why no new tests are needed for this patch.
          Also please list what manual steps were performed to verify this patch.

          +1 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 javadoc. There were no new javadoc warning messages.

          +1 eclipse:eclipse. The patch built with eclipse:eclipse.

          +1 findbugs. The patch does not introduce any new Findbugs (version 2.0.3) warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          +1 core tests. The patch passed unit tests in hadoop-common-project/hadoop-common.

          +1 contrib tests. The patch passed contrib unit tests.

          Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/8922//testReport/
          Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/8922//console

          This message is automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12685254/HDFS-7479-120414.patch against trunk revision 0653918. +1 @author . The patch does not contain any @author tags. -1 tests included . The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . There were no new javadoc warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. +1 core tests . The patch passed unit tests in hadoop-common-project/hadoop-common. +1 contrib tests . The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/8922//testReport/ Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/8922//console This message is automatically generated.
          Hide
          gtCarrera9 Li Lu added a comment -

          Hi Haohui Mai, in this patch I removed the deprecated o.a.h.fs.permission.AccessControlException, and updated o.a.h.security.AccessControlException class according to the removal. If you have time please feel free to take a look at it. Thanks!

          Show
          gtCarrera9 Li Lu added a comment - Hi Haohui Mai , in this patch I removed the deprecated o.a.h.fs.permission.AccessControlException, and updated o.a.h.security.AccessControlException class according to the removal. If you have time please feel free to take a look at it. Thanks!

            People

            • Assignee:
              gtCarrera9 Li Lu
              Reporter:
              wheat9 Haohui Mai
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development