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

incorrect result returned with binary protocol (prepared statements)

Details

    • 10.2.11, 10.2.12

    Description

      runnig following test suite with and without --ps-protocol produce different results.

      create table  DUPPP (code varchar(10) primary key);
      		 
      INSERT INTO DUPPP(code) VALUES ('LINE1'), ('LINE2'), ('LINE3');
      		 
      SELECT X.* FROM (SELECT CODE, RN FROM (SELECT A.CODE, @cnt := @cnt + 1 AS RN FROM  DUPPP A, (SELECT @cnt := 0) C) T) X;
      		 
      drop table DUPP;

      normal execution:

      CODE	RN
      		 
      LINE1	1
      		 
      LINE2	2
      		 
      LINE3	3

      with --ps-protocol:

      CODE	RN
      		 
      LINE3	3
      		 
      LINE3	3
      		 
      LINE3	3

      Attachments

        Issue Links

          relates to

          Bug - A problem which impairs or prevents the functions of the product. CONJ-338 Inconsistent/wrong resultset when using variables and nested selects

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Closed

          Activity

            Ascending order - Click to sort in descending order
            diego dupin Diego Dupin added a comment -

            to reproduced it easily on heidi with a query a little simpler:

            prepare test from "SELECT * FROM (SELECT * FROM (SELECT seq FROM seq_1_to_3, (SELECT @cnt := 0) C) T) X";
            execute test ;

            will be :

            3
            3
            3

            diego dupin Diego Dupin added a comment - to reproduced it easily on heidi with a query a little simpler: prepare test from "SELECT * FROM (SELECT * FROM (SELECT seq FROM seq_1_to_3, (SELECT @cnt := 0) C) T) X"; execute test ; will be : 3 3 3
            sanja Oleksandr Byelkin added a comment - - edited

            OK, sequences appeared later than 5.5, but the bug present in 5.5 so the first test suite sill in use

            sanja Oleksandr Byelkin added a comment - - edited OK, sequences appeared later than 5.5, but the bug present in 5.5 so the first test suite sill in use
            sanja Oleksandr Byelkin added a comment -

            there is no diferences in Item_func_set_user_var::update calls.

            sanja Oleksandr Byelkin added a comment - there is no diferences in Item_func_set_user_var::update calls.
            sanja Oleksandr Byelkin added a comment -

            The difference is that in binary protocol fields of table T instaed of X somehow used.

            sanja Oleksandr Byelkin added a comment - The difference is that in binary protocol fields of table T instaed of X somehow used.
            sanja Oleksandr Byelkin added a comment -

            Prepare decide to merge X, then execution to materialize X.

            sanja Oleksandr Byelkin added a comment - Prepare decide to merge X, then execution to materialize X.
            sanja Oleksandr Byelkin added a comment - - edited

            During statement prepare unit->prepared is set for X and so translation table created.

            sanja Oleksandr Byelkin added a comment - - edited During statement prepare unit->prepared is set for X and so translation table created.
            sanja Oleksandr Byelkin added a comment -

            Github tree: bb-5.5-MDEV-10657

            revision-id: 24b4585ad0648f21519f47a04762e1b208918749 (mariadb-5.5.58-18-g24b4585ad06)
            parent(s): 924db8b4ed3f268cbe91a1734611f4dc2311c7be
            author: Oleksandr Byelkin
            committer: Oleksandr Byelkin
            timestamp: 2017-12-20 13:52:27 +0100
            message:

            MDEV-10657: incorrect result returned with binary protocol (prepared statements)

            If translation table present when we materialize the derived table then
            change it to point on the materialized table.

            Added debug info to see really what happens with what derived.

            sanja Oleksandr Byelkin added a comment - Github tree: bb-5.5- MDEV-10657 revision-id: 24b4585ad0648f21519f47a04762e1b208918749 (mariadb-5.5.58-18-g24b4585ad06) parent(s): 924db8b4ed3f268cbe91a1734611f4dc2311c7be author: Oleksandr Byelkin committer: Oleksandr Byelkin timestamp: 2017-12-20 13:52:27 +0100 message: MDEV-10657 : incorrect result returned with binary protocol (prepared statements) If translation table present when we materialize the derived table then change it to point on the materialized table. Added debug info to see really what happens with what derived.
            igor Igor Babaev (Inactive) added a comment -

            Sanja, see your email with my review.

            igor Igor Babaev (Inactive) added a comment - Sanja, see your email with my review.

            People

              sanja Oleksandr Byelkin
              sanja Oleksandr Byelkin
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Git Integration

                  Error rendering 'com.xiplink.jira.git.jira_git_plugin:git-issue-webpanel'. Please contact your Jira administrators.