Details

    • Type: Improvement
    • Status: Closed
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: 3.0.0-beta1
    • Fix Version/s: 3.0.0-beta1
    • Component/s: httpfs
    • Labels:
      None

      Description

      While refactoring httpfs tests for HDFS-12052 I've noticed many style issues that are easy to fix, but should not be fixed in the same patch when we are fixing the bug in the code.

      I've been asked by at least two committers to create a separate patch which will only cover these trivial style fixes. So, here it is.

      1. HDFS-12193.patch
        18 kB
        Zoran Dimitrijevic

        Activity

        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #12051 (See https://builds.apache.org/job/Hadoop-trunk-Commit/12051/)
        HDFS-12193. Fix style issues in HttpFS tests. Contributed by Zoran (raviprak: rev c98201b5d83a700b4d081000065c6fd1a6ef2eed)

        • (edit) hadoop-hdfs-project/hadoop-hdfs-httpfs/src/test/java/org/apache/hadoop/fs/http/server/TestHttpFSServerNoXAttrs.java
        • (edit) hadoop-hdfs-project/hadoop-hdfs-httpfs/src/test/java/org/apache/hadoop/fs/http/server/TestHttpFSServer.java
        • (edit) hadoop-hdfs-project/hadoop-hdfs-httpfs/src/test/java/org/apache/hadoop/fs/http/server/TestHttpFSServerNoACLs.java
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #12051 (See https://builds.apache.org/job/Hadoop-trunk-Commit/12051/ ) HDFS-12193 . Fix style issues in HttpFS tests. Contributed by Zoran (raviprak: rev c98201b5d83a700b4d081000065c6fd1a6ef2eed) (edit) hadoop-hdfs-project/hadoop-hdfs-httpfs/src/test/java/org/apache/hadoop/fs/http/server/TestHttpFSServerNoXAttrs.java (edit) hadoop-hdfs-project/hadoop-hdfs-httpfs/src/test/java/org/apache/hadoop/fs/http/server/TestHttpFSServer.java (edit) hadoop-hdfs-project/hadoop-hdfs-httpfs/src/test/java/org/apache/hadoop/fs/http/server/TestHttpFSServerNoACLs.java
        Hide
        raviprak Ravi Prakash added a comment -

        Thanks for your contribution Zoran!

        Show
        raviprak Ravi Prakash added a comment - Thanks for your contribution Zoran!
        Hide
        raviprak Ravi Prakash added a comment -

        LGTM. +1. Committing shortly

        Show
        raviprak Ravi Prakash added a comment - LGTM. +1. Committing shortly

          People

          • Assignee:
            3opan Zoran Dimitrijevic
            Reporter:
            3opan Zoran Dimitrijevic
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Due:
              Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 1h
              1h
              Remaining:
              Remaining Estimate - 1h
              1h
              Logged:
              Time Spent - Not Specified
              Not Specified

                Development