Details

    • Type: Task Task
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.6.0
    • Component/s: None
    • Labels:

      Description

      I did some work under ACCUMULO-1599 to split up the existing integration tests that use MiniAccumuloCluster into two separate groups for testing: those integration tests that require custom configuration and need intimate knowledge and control of MiniAccumuloCluster, and those that don't.

      Those that do, were configured to use ConfigurableMacIT, and those that don't were configured to use SimpleMacIT.

      The intention was that any test that did not need to control MiniAccumuloCluster itself, and only used the regular Accumulo API for executing its tests, could use a shared instance, represented by SimpleMacIT. Ideally, we would start this shared instance during the pre-integration-test phase of the build lifecycle, and leave it running until the post-integration-test. If this instance was not already running, a JVM-local one could be instantiated for the test to run (eg. inside Eclipse).

      An additional benefit is that we could easily transition these tests, through a single, centralized configuration option, to run against a full instance (not MiniAccumuloCluster).

      However, something changed, and some commits against ACCUMULO-1009 reintroduced some of the complexity and exposed too much through SimpleMacIT.

      We need to document the intention and clean up this class to be simple again (with no knowledge of MiniAccumuloCluster underneath).

        Issue Links

          Activity

          Christopher Tubbs created issue -
          John Vines made changes -
          Field Original Value New Value
          Labels Documentation
          Christopher Tubbs made changes -
          Link This issue is related to ACCUMULO-2310 [ ACCUMULO-2310 ]
          Christopher Tubbs made changes -
          Fix Version/s 1.6.1 [ 12325441 ]
          Fix Version/s 1.6.0 [ 12322468 ]
          Hide
          ASF subversion and git services added a comment -

          Commit f4a9626a5a5e2a66c201218b13aeef558b63506c in accumulo's branch refs/heads/1.6.0-SNAPSHOT from Christopher Tubbs
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=f4a9626 ]

          ACCUMULO-1996 Add some javadocs to SimpleMacIT to define its scope

          Show
          ASF subversion and git services added a comment - Commit f4a9626a5a5e2a66c201218b13aeef558b63506c in accumulo's branch refs/heads/1.6.0-SNAPSHOT from Christopher Tubbs [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=f4a9626 ] ACCUMULO-1996 Add some javadocs to SimpleMacIT to define its scope
          Hide
          Christopher Tubbs added a comment -

          Added javadocs to declare the intended scope. Actual cleanup of the usage of this test class can be done in a separate issue which focuses on creating a more flexible API for testing against a running instance or MAC.

          Show
          Christopher Tubbs added a comment - Added javadocs to declare the intended scope. Actual cleanup of the usage of this test class can be done in a separate issue which focuses on creating a more flexible API for testing against a running instance or MAC.
          Christopher Tubbs made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Fix Version/s 1.6.0 [ 12322468 ]
          Fix Version/s 1.6.1 [ 12325441 ]
          Resolution Fixed [ 1 ]
          Hide
          ASF subversion and git services added a comment -

          Commit f4a9626a5a5e2a66c201218b13aeef558b63506c in accumulo's branch refs/heads/master from Christopher Tubbs
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=f4a9626 ]

          ACCUMULO-1996 Add some javadocs to SimpleMacIT to define its scope

          Show
          ASF subversion and git services added a comment - Commit f4a9626a5a5e2a66c201218b13aeef558b63506c in accumulo's branch refs/heads/master from Christopher Tubbs [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=f4a9626 ] ACCUMULO-1996 Add some javadocs to SimpleMacIT to define its scope
          Christopher Tubbs made changes -
          Link This issue is related to ACCUMULO-3078 [ ACCUMULO-3078 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          111d 19h 29m 1 Christopher Tubbs 31/Mar/14 18:42

            People

            • Assignee:
              Christopher Tubbs
              Reporter:
              Christopher Tubbs
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development