Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • 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. azure-auth-keys.xml
          3 kB
          Sean Mackrory
        2. HADOOP-15546-001.patch
          54 kB
          Steve Loughran
        3. HADOOP-15546-HADOOP-15407-001.patch
          54 kB
          Steve Loughran
        4. HADOOP-15546-HADOOP-15407-002.patch
          102 kB
          Steve Loughran
        5. HADOOP-15546-HADOOP-15407-003.patch
          110 kB
          Steve Loughran
        6. HADOOP-15546-HADOOP-15407-004.patch
          110 kB
          Steve Loughran
        7. HADOOP-15546-HADOOP-15407-005.patch
          253 kB
          Steve Loughran
        8. HADOOP-15546-HADOOP-15407-006.patch
          254 kB
          Steve Loughran
        9. HADOOP-15546-HADOOP-15407-006.patch
          254 kB
          Steve Loughran
        10. HADOOP-15546-HADOOP-15407-007.patch
          263 kB
          Steve Loughran
        11. HADOOP-15546-HADOOP-15407-008.patch
          276 kB
          Steve Loughran
        12. HADOOP-15546-HADOOP-15407-009.patch
          276 kB
          Steve Loughran
        13. HADOOP-15546-HADOOP-15407-010.patch
          269 kB
          Da Zhou
        14. HADOOP-15546-HADOOP-15407-011.patch
          276 kB
          Steve Loughran
        15. HADOOP-15546-HADOOP-15407-012.patch
          284 kB
          Da Zhou

          Issue Links

            Activity

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: