Uploaded image for project: 'IMPALA'
  1. IMPALA
  2. IMPALA-4117

Factor out test utility classes from simple-scheduler-test.cc

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: Impala 2.7.0
    • Fix Version/s: Impala 2.8.0
    • Component/s: Backend
    • Labels:
      None

      Description

      As a prerequisite to IMPALA-4086 we need to move the test utility classes from simple-scheduler-test.cc into their own .h/.cc files.

        Activity

        Hide
        lv Lars Volker added a comment -

        IMPALA-4117: Factor simple scheduler test code into own files

        This change merely splits the helper classes from
        simple-scheduler-test.cc into separate .h and .cc files. It does not
        change the semantics of the code. Whitespace formatting has been done
        with git-clang-format for any changed lines.

        Change-Id: Id3a6b3336db175eb095cbeb7ec623a5957b77ccc
        Reviewed-on: http://gerrit.cloudera.org:8080/4486
        Reviewed-by: Matthew Jacobs <mj@cloudera.com>
        Reviewed-by: Dan Hecht <dhecht@cloudera.com>
        Tested-by: Internal Jenkins

        Show
        lv Lars Volker added a comment - IMPALA-4117 : Factor simple scheduler test code into own files This change merely splits the helper classes from simple-scheduler-test.cc into separate .h and .cc files. It does not change the semantics of the code. Whitespace formatting has been done with git-clang-format for any changed lines. Change-Id: Id3a6b3336db175eb095cbeb7ec623a5957b77ccc Reviewed-on: http://gerrit.cloudera.org:8080/4486 Reviewed-by: Matthew Jacobs <mj@cloudera.com> Reviewed-by: Dan Hecht <dhecht@cloudera.com> Tested-by: Internal Jenkins

          People

          • Assignee:
            lv Lars Volker
            Reporter:
            lv Lars Volker
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development