Details

    • Type: Task
    • Status: Resolved
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: Impala 2.8.0
    • Fix Version/s: Impala 2.8.0
    • Component/s: Infrastructure
    • Labels:

      Description

      Tracks improvements to the Kudu DML functional tests.

        Activity

        Hide
        twmarshall Thomas Tauber-Marshall added a comment -

        commit 3833707dbd78dd39f6cf25060607a7b8ae9bff10
        Author: Thomas Tauber-Marshall <tmarshall@cloudera.com>
        Date: Fri Nov 4 12:52:07 2016 -0700

        IMPALA-4466: Improve Kudu CRUD test coverage

        The results in the test files were verified by hand.

        This patch also introduces a new test section 'DML_RESULTS', which
        takes the name of a table as a comment and the contents of the
        table as its body and then verifies that the body matches the
        actual contents of the table. This makes it easy to check that a
        DML operation has the desired effect on the contents of a table,
        rather than always having to add another test case that runs a
        select on the table. For now, this section cannot be used in a
        test along with the RESULTS or ERRORS sections.

        TODO: Refactor the DML test case handling (IMPALA-4471)

        Change-Id: Ib9e7afbef60186edb00a9d11fbe5a8c64931add6
        Reviewed-on: http://gerrit.cloudera.org:8080/4953
        Reviewed-by: Matthew Jacobs <mj@cloudera.com>
        Tested-by: Internal Jenkins

        Show
        twmarshall Thomas Tauber-Marshall added a comment - commit 3833707dbd78dd39f6cf25060607a7b8ae9bff10 Author: Thomas Tauber-Marshall <tmarshall@cloudera.com> Date: Fri Nov 4 12:52:07 2016 -0700 IMPALA-4466 : Improve Kudu CRUD test coverage The results in the test files were verified by hand. This patch also introduces a new test section 'DML_RESULTS', which takes the name of a table as a comment and the contents of the table as its body and then verifies that the body matches the actual contents of the table. This makes it easy to check that a DML operation has the desired effect on the contents of a table, rather than always having to add another test case that runs a select on the table. For now, this section cannot be used in a test along with the RESULTS or ERRORS sections. TODO: Refactor the DML test case handling ( IMPALA-4471 ) Change-Id: Ib9e7afbef60186edb00a9d11fbe5a8c64931add6 Reviewed-on: http://gerrit.cloudera.org:8080/4953 Reviewed-by: Matthew Jacobs <mj@cloudera.com> Tested-by: Internal Jenkins

          People

          • Assignee:
            twmarshall Thomas Tauber-Marshall
            Reporter:
            mjacobs Matthew Jacobs
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development