Hadoop Common
  1. Hadoop Common
  2. HADOOP-7509

Improve message when Authentication is required

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 0.20.204.0, 0.23.0
    • Fix Version/s: 1.1.0, 0.23.0, 0.24.0
    • Component/s: None
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      The message when security is enabled and authentication is configured to be simple is not explicit enough. It simply prints out "Authentication is required" and prints out a stack trace. The message should be "Authorization (hadoop.security.authorization) is enabled but authentication (hadoop.security.authentication) is configured as simple. Please configure another method."

      1. correct7509.branch-0.23.patch
        1 kB
        Ravi Prakash
      2. HADOOP-7509.1.patch
        1 kB
        Ravi Prakash
      3. HADOOP-7509.2.branch-0.23.patch
        1 kB
        Ravi Prakash
      4. HADOOP-7509.2.patch
        1 kB
        Ravi Prakash
      5. patchToCorrect7509.branch-0.20.patch
        0.9 kB
        Ravi Prakash
      6. patchToCorrect7509.patch
        1 kB
        Ravi Prakash

        Issue Links

          Activity

          Hide
          Matt Foley added a comment -

          Moved spurious Release Note to comment:

          Thanks Aaron and Suresh! Marking as resolved fixed since changes have gone in.

          Show
          Matt Foley added a comment - Moved spurious Release Note to comment: Thanks Aaron and Suresh! Marking as resolved fixed since changes have gone in.
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Hdfs-trunk #846 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/846/)
          Move HADOOP-7668, HADOOP-7509, HADOOP-7745, MAPREDUCE-2764, HADOOP-7766 and HADOOP-7721 entries from trunk to 0.23 in CHANGES.txt.

          szetszwo : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1190132
          Files :

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Show
          Hudson added a comment - Integrated in Hadoop-Hdfs-trunk #846 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/846/ ) Move HADOOP-7668 , HADOOP-7509 , HADOOP-7745 , MAPREDUCE-2764 , HADOOP-7766 and HADOOP-7721 entries from trunk to 0.23 in CHANGES.txt. szetszwo : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1190132 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Hdfs-0.23-Build #53 (See https://builds.apache.org/job/Hadoop-Hdfs-0.23-Build/53/)
          svn merge -c 1182641 from trunk for HADOOP-7509.

          szetszwo : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1189986
          Files :

          • /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common
          • /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/java
          • /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Show
          Hudson added a comment - Integrated in Hadoop-Hdfs-0.23-Build #53 (See https://builds.apache.org/job/Hadoop-Hdfs-0.23-Build/53/ ) svn merge -c 1182641 from trunk for HADOOP-7509 . szetszwo : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1189986 Files : /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/java /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-trunk #876 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/876/)
          Move HADOOP-7668, HADOOP-7509, HADOOP-7745, MAPREDUCE-2764, HADOOP-7766 and HADOOP-7721 entries from trunk to 0.23 in CHANGES.txt.

          szetszwo : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1190132
          Files :

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk #876 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/876/ ) Move HADOOP-7668 , HADOOP-7509 , HADOOP-7745 , MAPREDUCE-2764 , HADOOP-7766 and HADOOP-7721 entries from trunk to 0.23 in CHANGES.txt. szetszwo : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1190132 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Common-trunk-Commit #1182 (See https://builds.apache.org/job/Hadoop-Common-trunk-Commit/1182/)
          Move HADOOP-7668, HADOOP-7509, HADOOP-7745, MAPREDUCE-2764, HADOOP-7766 and HADOOP-7721 entries from trunk to 0.23 in CHANGES.txt.

          szetszwo : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1190132
          Files :

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Show
          Hudson added a comment - Integrated in Hadoop-Common-trunk-Commit #1182 (See https://builds.apache.org/job/Hadoop-Common-trunk-Commit/1182/ ) Move HADOOP-7668 , HADOOP-7509 , HADOOP-7745 , MAPREDUCE-2764 , HADOOP-7766 and HADOOP-7721 entries from trunk to 0.23 in CHANGES.txt. szetszwo : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1190132 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Hdfs-trunk-Commit #1259 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Commit/1259/)
          Move HADOOP-7668, HADOOP-7509, HADOOP-7745, MAPREDUCE-2764, HADOOP-7766 and HADOOP-7721 entries from trunk to 0.23 in CHANGES.txt.

          szetszwo : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1190132
          Files :

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Show
          Hudson added a comment - Integrated in Hadoop-Hdfs-trunk-Commit #1259 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Commit/1259/ ) Move HADOOP-7668 , HADOOP-7509 , HADOOP-7745 , MAPREDUCE-2764 , HADOOP-7766 and HADOOP-7721 entries from trunk to 0.23 in CHANGES.txt. szetszwo : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1190132 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-trunk-Commit #1201 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Commit/1201/)
          Move HADOOP-7668, HADOOP-7509, HADOOP-7745, MAPREDUCE-2764, HADOOP-7766 and HADOOP-7721 entries from trunk to 0.23 in CHANGES.txt.

          szetszwo : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1190132
          Files :

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk-Commit #1201 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Commit/1201/ ) Move HADOOP-7668 , HADOOP-7509 , HADOOP-7745 , MAPREDUCE-2764 , HADOOP-7766 and HADOOP-7721 entries from trunk to 0.23 in CHANGES.txt. szetszwo : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1190132 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-0.23-Build #65 (See https://builds.apache.org/job/Hadoop-Mapreduce-0.23-Build/65/)
          svn merge -c 1182641 from trunk for HADOOP-7509.

          szetszwo : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1189986
          Files :

          • /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common
          • /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/java
          • /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-0.23-Build #65 (See https://builds.apache.org/job/Hadoop-Mapreduce-0.23-Build/65/ ) svn merge -c 1182641 from trunk for HADOOP-7509 . szetszwo : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1189986 Files : /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/java /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-0.23-Commit #77 (See https://builds.apache.org/job/Hadoop-Mapreduce-0.23-Commit/77/)
          svn merge -c 1182641 from trunk for HADOOP-7509.

          szetszwo : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1189986
          Files :

          • /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common
          • /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/java
          • /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-0.23-Commit #77 (See https://builds.apache.org/job/Hadoop-Mapreduce-0.23-Commit/77/ ) svn merge -c 1182641 from trunk for HADOOP-7509 . szetszwo : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1189986 Files : /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/java /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Common-0.23-Commit #81 (See https://builds.apache.org/job/Hadoop-Common-0.23-Commit/81/)
          svn merge -c 1182641 from trunk for HADOOP-7509.

          szetszwo : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1189986
          Files :

          • /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common
          • /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/java
          • /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Show
          Hudson added a comment - Integrated in Hadoop-Common-0.23-Commit #81 (See https://builds.apache.org/job/Hadoop-Common-0.23-Commit/81/ ) svn merge -c 1182641 from trunk for HADOOP-7509 . szetszwo : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1189986 Files : /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/java /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Hdfs-0.23-Commit #81 (See https://builds.apache.org/job/Hadoop-Hdfs-0.23-Commit/81/)
          svn merge -c 1182641 from trunk for HADOOP-7509.

          szetszwo : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1189986
          Files :

          • /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common
          • /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/java
          • /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Show
          Hudson added a comment - Integrated in Hadoop-Hdfs-0.23-Commit #81 (See https://builds.apache.org/job/Hadoop-Hdfs-0.23-Commit/81/ ) svn merge -c 1182641 from trunk for HADOOP-7509 . szetszwo : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1189986 Files : /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/java /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Hide
          Tsz Wo Nicholas Sze added a comment -

          Merged to 0.23.

          Show
          Tsz Wo Nicholas Sze added a comment - Merged to 0.23.
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-trunk #860 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/860/)
          HADOOP-7545. Fix wrong variable name in exception message introduced in HADOOP-7509. Contributed by Ravi Prakash.

          suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1183132
          Files :

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk #860 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/860/ ) HADOOP-7545 . Fix wrong variable name in exception message introduced in HADOOP-7509 . Contributed by Ravi Prakash. suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1183132 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Hdfs-trunk #830 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/830/)
          HADOOP-7545. Fix wrong variable name in exception message introduced in HADOOP-7509. Contributed by Ravi Prakash.

          suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1183132
          Files :

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Show
          Hudson added a comment - Integrated in Hadoop-Hdfs-trunk #830 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/830/ ) HADOOP-7545 . Fix wrong variable name in exception message introduced in HADOOP-7509 . Contributed by Ravi Prakash. suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1183132 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-trunk-Commit #1094 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Commit/1094/)
          HADOOP-7545. Fix wrong variable name in exception message introduced in HADOOP-7509. Contributed by Ravi Prakash.

          suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1183132
          Files :

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk-Commit #1094 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Commit/1094/ ) HADOOP-7545 . Fix wrong variable name in exception message introduced in HADOOP-7509 . Contributed by Ravi Prakash. suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1183132 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Common-trunk-Commit #1075 (See https://builds.apache.org/job/Hadoop-Common-trunk-Commit/1075/)
          HADOOP-7545. Fix wrong variable name in exception message introduced in HADOOP-7509. Contributed by Ravi Prakash.

          suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1183132
          Files :

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Show
          Hudson added a comment - Integrated in Hadoop-Common-trunk-Commit #1075 (See https://builds.apache.org/job/Hadoop-Common-trunk-Commit/1075/ ) HADOOP-7545 . Fix wrong variable name in exception message introduced in HADOOP-7509 . Contributed by Ravi Prakash. suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1183132 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Hdfs-trunk-Commit #1153 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Commit/1153/)
          HADOOP-7545. Fix wrong variable name in exception message introduced in HADOOP-7509. Contributed by Ravi Prakash.

          suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1183132
          Files :

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Show
          Hudson added a comment - Integrated in Hadoop-Hdfs-trunk-Commit #1153 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Commit/1153/ ) HADOOP-7545 . Fix wrong variable name in exception message introduced in HADOOP-7509 . Contributed by Ravi Prakash. suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1183132 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Hide
          Aaron T. Myers added a comment -

          Thanks for pointing that out Aaron! I'm terribly sorry for the silly error.

          No biggie. Innocent mistake.

          Show
          Aaron T. Myers added a comment - Thanks for pointing that out Aaron! I'm terribly sorry for the silly error. No biggie. Innocent mistake.
          Hide
          Suresh Srinivas added a comment -

          Thanks for catching that. I should have caught it. Had to read your comment twice to see the switch

          Show
          Suresh Srinivas added a comment - Thanks for catching that. I should have caught it. Had to read your comment twice to see the switch
          Hide
          Ravi Prakash added a comment -

          This patch applies to current branch-0.23 (when the earlier wrong patch has not yet been merged into branch-0.23)

          Show
          Ravi Prakash added a comment - This patch applies to current branch-0.23 (when the earlier wrong patch has not yet been merged into branch-0.23)
          Hide
          Ravi Prakash added a comment -

          This fixes the switcheroo on branch-0.20-security.

          Show
          Ravi Prakash added a comment - This fixes the switcheroo on branch-0.20-security.
          Hide
          Ravi Prakash added a comment -

          Switched between CommonConfigurationKeys.HADOOP_SECURITY_AUTHENTICATION and CommonConfigurationKeys.HADOOP_SECURITY_AUTHORIZATION

          Show
          Ravi Prakash added a comment - Switched between CommonConfigurationKeys.HADOOP_SECURITY_AUTHENTICATION and CommonConfigurationKeys.HADOOP_SECURITY_AUTHORIZATION
          Hide
          Ravi Prakash added a comment -

          Thanks for pointing that out Aaron! I'm terribly sorry for the silly error.

          I'm attaching patchToCorrect7509.patch which goes on top of what's already committed to trunk.

          Show
          Ravi Prakash added a comment - Thanks for pointing that out Aaron! I'm terribly sorry for the silly error. I'm attaching patchToCorrect7509.patch which goes on top of what's already committed to trunk.
          Hide
          Aaron T. Myers added a comment -

          The error message introduced in this patch appears to reverse the references to the configuration keys:

          AccessControlException ae = new AccessControlException("Authorization ("
            + CommonConfigurationKeys.HADOOP_SECURITY_AUTHENTICATION
            + ") is enabled but authentication ("
            + CommonConfigurationKeys.HADOOP_SECURITY_AUTHORIZATION
            + ") is configured as simple. Please configure another method "
            + "like kerberos or digest.");
          

          Could someone please either revert this patch from the branches where it was committed or open a new JIRA to address this issue ASAP?

          Show
          Aaron T. Myers added a comment - The error message introduced in this patch appears to reverse the references to the configuration keys: AccessControlException ae = new AccessControlException( "Authorization (" + CommonConfigurationKeys.HADOOP_SECURITY_AUTHENTICATION + ") is enabled but authentication (" + CommonConfigurationKeys.HADOOP_SECURITY_AUTHORIZATION + ") is configured as simple. Please configure another method " + "like kerberos or digest." ); Could someone please either revert this patch from the branches where it was committed or open a new JIRA to address this issue ASAP?
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Hdfs-trunk #829 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/829/)
          HADOOP-7509. Improve exception message thrown when Authentication is required. Contributed by Ravi Prakash.

          suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1182641
          Files :

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Show
          Hudson added a comment - Integrated in Hadoop-Hdfs-trunk #829 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/829/ ) HADOOP-7509 . Improve exception message thrown when Authentication is required. Contributed by Ravi Prakash. suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1182641 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-trunk #859 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/859/)
          HADOOP-7509. Improve exception message thrown when Authentication is required. Contributed by Ravi Prakash.

          suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1182641
          Files :

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk #859 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/859/ ) HADOOP-7509 . Improve exception message thrown when Authentication is required. Contributed by Ravi Prakash. suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1182641 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-trunk-Commit #1086 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Commit/1086/)
          HADOOP-7509. Improve exception message thrown when Authentication is required. Contributed by Ravi Prakash.

          suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1182641
          Files :

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk-Commit #1086 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Commit/1086/ ) HADOOP-7509 . Improve exception message thrown when Authentication is required. Contributed by Ravi Prakash. suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1182641 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Hdfs-trunk-Commit #1144 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Commit/1144/)
          HADOOP-7509. Improve exception message thrown when Authentication is required. Contributed by Ravi Prakash.

          suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1182641
          Files :

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Show
          Hudson added a comment - Integrated in Hadoop-Hdfs-trunk-Commit #1144 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Commit/1144/ ) HADOOP-7509 . Improve exception message thrown when Authentication is required. Contributed by Ravi Prakash. suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1182641 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Common-trunk-Commit #1066 (See https://builds.apache.org/job/Hadoop-Common-trunk-Commit/1066/)
          HADOOP-7509. Improve exception message thrown when Authentication is required. Contributed by Ravi Prakash.

          suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1182641
          Files :

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Show
          Hudson added a comment - Integrated in Hadoop-Common-trunk-Commit #1066 (See https://builds.apache.org/job/Hadoop-Common-trunk-Commit/1066/ ) HADOOP-7509 . Improve exception message thrown when Authentication is required. Contributed by Ravi Prakash. suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1182641 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
          Hide
          Suresh Srinivas added a comment -

          I committed the patch to 0.20.206 and trunk. Steve, please open another issue, for attaching wiki URL to the exception.

          Show
          Suresh Srinivas added a comment - I committed the patch to 0.20.206 and trunk. Steve, please open another issue, for attaching wiki URL to the exception.
          Hide
          Suresh Srinivas added a comment -

          +1 for the patch.

          Show
          Suresh Srinivas added a comment - +1 for the patch.
          Hide
          Steve Loughran added a comment -

          This looks good, and I can see why no tests are needed.

          One possible improvement -which I've started doing with connection errors -is have the exception actually point to a hadoop wiki page entry where more detailed diagnostics and troubleshooting hints can go in, such as for http://wiki.apache.org/hadoop/NoRouteToHost . Would that make sense here -and is there already a wiki page to point to?

          Show
          Steve Loughran added a comment - This looks good, and I can see why no tests are needed. One possible improvement -which I've started doing with connection errors -is have the exception actually point to a hadoop wiki page entry where more detailed diagnostics and troubleshooting hints can go in, such as for http://wiki.apache.org/hadoop/NoRouteToHost . Would that make sense here -and is there already a wiki page to point to?
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12498204/HADOOP-7509.2.branch-0.23.patch
          against trunk revision .

          +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 javadoc. The javadoc tool did not generate any warning messages.

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

          +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) 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 .

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

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

          This message is automatically generated.

          Show
          Hadoop QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12498204/HADOOP-7509.2.branch-0.23.patch against trunk revision . +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 javadoc. The javadoc tool did not generate any warning messages. +1 javac. The applied patch does not increase the total number of javac compiler warnings. +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) 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 . +1 contrib tests. The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/276//testReport/ Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/276//console This message is automatically generated.
          Hide
          Ravi Prakash added a comment -

          This patch applies to trunk and branch-0.23

          Show
          Ravi Prakash added a comment - This patch applies to trunk and branch-0.23
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12493039/HADOOP-7509.2.patch
          against trunk revision .

          +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 patch. The patch command could not apply the patch.

          Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/135//console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12493039/HADOOP-7509.2.patch against trunk revision . +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 patch. The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/135//console This message is automatically generated.
          Hide
          Ravi Prakash added a comment -

          Hi Harsh,

          Thanks for the review. I've changed the message to mention kerberos or digest. If you had more in mind, please let me know.

          Show
          Ravi Prakash added a comment - Hi Harsh, Thanks for the review. I've changed the message to mention kerberos or digest. If you had more in mind, please let me know.
          Hide
          Harsh J added a comment -

          Hey Ravi,

          Thanks for this improvement!

          Personally, I think "Please configure another method." can still be elaborated a bit if we're gonna get helpful with error messages anyway

          Show
          Harsh J added a comment - Hey Ravi, Thanks for this improvement! Personally, I think "Please configure another method." can still be elaborated a bit if we're gonna get helpful with error messages anyway
          Hide
          Ravi Prakash added a comment -

          The log message looks like

          2011-04-04 17:52:48,916 INFO org.apache.hadoop.ipc.Server: IPC Server listener on 8020: readAndProcess threw exception
          org.apache.hadoop.security.AccessControlException: Authentication is required. Count of bytes read: 0
          org.apache.hadoop.security.AccessControlException: Authentication is required
                  at org.apache.hadoop.ipc.Server$Connection.readAndProcess(Server.java:1121)
                  at org.apache.hadoop.ipc.Server$Listener.doRead(Server.java:536)
                  at org.apache.hadoop.ipc.Server$Listener$Reader.run(Server.java:343)
                  at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
                  at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
                  at java.lang.Thread.run(Thread.java:619)
          

          This confuses users. The error can be more detailed. Attaching a patch to do so. Can someone please review and commit it? No tests attached because its too small a change.

          Show
          Ravi Prakash added a comment - The log message looks like 2011-04-04 17:52:48,916 INFO org.apache.hadoop.ipc.Server: IPC Server listener on 8020: readAndProcess threw exception org.apache.hadoop.security.AccessControlException: Authentication is required. Count of bytes read: 0 org.apache.hadoop.security.AccessControlException: Authentication is required at org.apache.hadoop.ipc.Server$Connection.readAndProcess(Server.java:1121) at org.apache.hadoop.ipc.Server$Listener.doRead(Server.java:536) at org.apache.hadoop.ipc.Server$Listener$Reader.run(Server.java:343) at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886) at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908) at java.lang.Thread.run(Thread.java:619) This confuses users. The error can be more detailed. Attaching a patch to do so. Can someone please review and commit it? No tests attached because its too small a change.

            People

            • Assignee:
              Ravi Prakash
              Reporter:
              Ravi Prakash
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development