Uploaded image for project: 'MariaDB Server'
  1. MariaDB Server
  2. MDEV-6958 10.1 testing
  3. MDEV-6923

Testing for SET STATEMENT .. FOR (MDEV-5231, per-query variables)

    Details

    • Type: Technical task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: N/A
    • Component/s: Tests
    • Labels:

      Gliffy Diagrams

        Attachments

          Issue Links

            Activity

            Hide
            elenst Elena Stepanova added a comment - - edited

            Currently in the "standard" configuration (Linux, normal server with XtraDB, no additional plugins) we have 148 variables which are not read-only and have session scope.
            As of

            bb-10.1-set-statement
            commit d6981391eb8f904135dfdedb5b90a7592b35e648
            Author: Oleksandr Byelkin <sanja@mariadb.com>
            Date:   Thu Oct 30 19:34:54 2014 +0100
            

            10 are already disabled for SET STATEMENT (cause ER_SET_STATEMENT_NOT_SUPPORTED).

            From the remaining ones, I tried about a half – not like tested them deeply, but at least checked that they do something more than just display their value in a SELECT under SET STATEMENT ... FOR.
            Out of this half, about 2/3 worked and 1/3 didn't.
            This gives enough information for further interpolation.

            Bugs for discovered failures have been filed.
            Most of failures are reproducible with Percona server, but not all of them.
            Bugs reproducible on Percona server are also filed in their LP.

            Attached is a rough test case for all variables (including read-only, global and unsupported). The windows version is the same as Linux minus several variables that do not exist on Windows.

            Show
            elenst Elena Stepanova added a comment - - edited Currently in the "standard" configuration (Linux, normal server with XtraDB, no additional plugins) we have 148 variables which are not read-only and have session scope. As of bb-10.1-set-statement commit d6981391eb8f904135dfdedb5b90a7592b35e648 Author: Oleksandr Byelkin <sanja@mariadb.com> Date: Thu Oct 30 19:34:54 2014 +0100 10 are already disabled for SET STATEMENT (cause ER_SET_STATEMENT_NOT_SUPPORTED). From the remaining ones, I tried about a half – not like tested them deeply, but at least checked that they do something more than just display their value in a SELECT under SET STATEMENT ... FOR. Out of this half, about 2/3 worked and 1/3 didn't. This gives enough information for further interpolation. Bugs for discovered failures have been filed. Most of failures are reproducible with Percona server, but not all of them. Bugs reproducible on Percona server are also filed in their LP. Attached is a rough test case for all variables (including read-only, global and unsupported). The windows version is the same as Linux minus several variables that do not exist on Windows.
            Hide
            elenst Elena Stepanova added a comment - - edited

            There are also plugin variables.

            Query_response_time comes with 1, it doesn't work.
            Spider comes with 80, didn't check them.
            Test_sql_discovery comes with 2, they work.
            Example comes with 4, didn't check them.
            simple_parser comes with 2, didn't check them.
            TokuDB comes with 28, didn't check them.

            Didn't check Cassandra, Mroonga, probably more.

            Show
            elenst Elena Stepanova added a comment - - edited There are also plugin variables. Query_response_time comes with 1, it doesn't work. Spider comes with 80, didn't check them. Test_sql_discovery comes with 2, they work. Example comes with 4, didn't check them. simple_parser comes with 2, didn't check them. TokuDB comes with 28, didn't check them. Didn't check Cassandra, Mroonga, probably more.
            Hide
            elenst Elena Stepanova added a comment -

            Added new Linux test (the Windows one is not updated).

            Show
            elenst Elena Stepanova added a comment - Added new Linux test (the Windows one is not updated).
            Hide
            elenst Elena Stepanova added a comment -

            The feature has not been fully tested, but it's been tested reasonably in accordance to its relative importance.

            Show
            elenst Elena Stepanova added a comment - The feature has not been fully tested, but it's been tested reasonably in accordance to its relative importance.

              People

              • Assignee:
                elenst Elena Stepanova
                Reporter:
                elenst Elena Stepanova
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 1 week, 2 hours
                  1w 2h