Uploaded image for project: 'MariaDB Server'
  1. MariaDB Server
  2. MDEV-4546

[PATCH] Perfschema unit tests to return non-zero on failure

        Details

        • Type: Bug
        • Status: Closed
        • Priority: Major
        • Resolution: Fixed
        • Affects Version/s: 10.0.2
        • Fix Version/s: 10.0.4
        • Component/s: None
        • Labels:
          None

          Description

          Please consider including the attached patch.
          It fixes perfschema unit tests to return non-zero on failure which will make them to be actually considered failed by make when they fail.

          Also it adds one missed unit test into the build.

            Gliffy Diagrams

              Attachments

                Activity

                Ascending order - Click to sort in descending order
                Hide
                Permalink
                sanja Oleksandr Byelkin added a comment -

                Pavel, under what license do we have these patches (as well as patch in 4547) ? If it's new BSD, could you say that explicitly, please?

                Show
                sanja Oleksandr Byelkin added a comment - Pavel, under what license do we have these patches (as well as patch in 4547) ? If it's new BSD, could you say that explicitly, please?
                Hide
                Permalink
                sanja Oleksandr Byelkin added a comment -

                The patch is OK (waiting info about license)

                Show
                sanja Oleksandr Byelkin added a comment - The patch is OK (waiting info about license)
                Hide
                Permalink
                sanja Oleksandr Byelkin added a comment -

                thanks! I saw that it is new BSD

                Show
                sanja Oleksandr Byelkin added a comment - thanks! I saw that it is new BSD
                Hide
                Permalink
                sanja Oleksandr Byelkin added a comment -

                Pushed to 10.0

                Show
                sanja Oleksandr Byelkin added a comment - Pushed to 10.0

                  People

                  • Assignee:
                    sanja Oleksandr Byelkin
                    Reporter:
                    pivanof Pavel Ivanov
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    3 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: