VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Sub-task
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 2.9.0, 3.0.0
    • 3.2.0, 3.1.4, 2.10.1
    • fs/s3
    • None

    Description

      I'm going to propose cutting out the localdynamo test option for s3guard

      • the local DDB JAR is unmaintained/lags the SDK We work with...eventually there'll be differences in API.
      • as the local dynamo DB is unshaded. it complicates classpath setup for the build. Remove it and there's no need to worry about versions of anything other than the shaded AWS
      • it complicates test runs. Now we need to test for both localdynamo and real dynamo
      • but we can't ignore real dynamo, because that's the one which matters

      While the local option promises to reduce test costs, really, it's just adding complexity. If you are testing with s3guard, you need to have a real table to test against., And with the exception of those people testing s3a against non-AWS, consistent endpoints, everyone should be testing with S3Guard.

      Straightforward to remove.

      Attachments

        1. HADOOP-14918.005.patch
          49 kB
          Gabor Bota
        2. HADOOP-14918.006.patch
          49 kB
          Gabor Bota
        3. HADOOP-14918-001.patch
          39 kB
          Steve Loughran
        4. HADOOP-14918-002.patch
          39 kB
          Steve Loughran
        5. HADOOP-14918-003.patch
          34 kB
          Steve Loughran
        6. HADOOP-14918-004.patch
          44 kB
          Gabor Bota
        7. HADOOP-14918-branch-2.10.001.patch
          39 kB
          Jonathan Hung

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            gabor.bota Gabor Bota
            stevel@apache.org Steve Loughran
            Votes:
            0 Vote for this issue
            Watchers:
            8 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment