Details

    • Type: Sub-task
    • Status: Open
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: HADOOP-15407
    • Fix Version/s: None
    • Component/s: fs/azure
    • Labels:
      None

      Description

      Followup on HADOOP-15540 with some initial review tuning

      Tuning

      • ordering of imports
      • rely on azure-auth-keys.xml to store credentials (change imports, docs,.gitignore)
      • log4j -> info
      • add a "." to the first sentence of all the javadocs I noticed.
      • remove @Public annotations except for some constants (which includes some commitment to maintain them).
      • move the AbstractFS declarations out of the src/test/resources XML file into core-default.xml for all to use
      • other IDE-suggested tweaks

      Testing

      Review the tests, move to ContractTestUtil assertions, make more consistent to contract test setup, and general work to make the tests work well over slower links, document, etc.

        Attachments

        1. HADOOP-15546-001.patch
          54 kB
          Steve Loughran
        2. HADOOP-15546-HADOOP-15407-001.patch
          54 kB
          Steve Loughran
        3. HADOOP-15546-HADOOP-15407-002.patch
          102 kB
          Steve Loughran
        4. HADOOP-15546-HADOOP-15407-003.patch
          110 kB
          Steve Loughran
        5. HADOOP-15546-HADOOP-15407-004.patch
          110 kB
          Steve Loughran
        6. HADOOP-15546-HADOOP-15407-005.patch
          253 kB
          Steve Loughran
        7. HADOOP-15546-HADOOP-15407-006.patch
          254 kB
          Steve Loughran
        8. HADOOP-15546-HADOOP-15407-006.patch
          254 kB
          Steve Loughran

          Issue Links

            Activity

              People

              • Assignee:
                stevel@apache.org Steve Loughran
                Reporter:
                stevel@apache.org Steve Loughran
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated: