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

MariaDB 5.5.40 server crash on update table left join with a view

    Details

      Description

      Several crashes reported on update table left join with a view

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              sanja Oleksandr Byelkin added a comment -

              Sent for additional testing

              Show
              sanja Oleksandr Byelkin added a comment - Sent for additional testing
              Hide
              elenst Elena Stepanova added a comment - - edited

              Regarding the patch:
              it didn't apply smoothly due to line breaks in long lines in the test case. I removed them (locally) and then it worked.

              Regarding the test case:
              It creates InnoDB tables, but InnoDB is disabled for the test case. Please either enable it or switch tables to MyISAM, it just doesn't make sense to generate warnings about the missing engine.

              Otherwise, I've run a bunch of tests, didn't get any regressions. Please push.

              Show
              elenst Elena Stepanova added a comment - - edited Regarding the patch: it didn't apply smoothly due to line breaks in long lines in the test case. I removed them (locally) and then it worked. Regarding the test case: It creates InnoDB tables, but InnoDB is disabled for the test case. Please either enable it or switch tables to MyISAM, it just doesn't make sense to generate warnings about the missing engine. Otherwise, I've run a bunch of tests, didn't get any regressions. Please push.
              Hide
              danblack Daniel Black added a comment -
              Show
              danblack Daniel Black added a comment - for merging: https://github.com/MariaDB/server/pull/40
              Hide
              sanja Oleksandr Byelkin added a comment -

              It is good, but if engine is not important it is better do not mention it at all (my fault)

              Show
              sanja Oleksandr Byelkin added a comment - It is good, but if engine is not important it is better do not mention it at all (my fault)
              Hide
              sanja Oleksandr Byelkin added a comment - - edited

              Daniel, thank you a lot, but I think more changes should be done so I've made changes from my tree and push after test run.

              Show
              sanja Oleksandr Byelkin added a comment - - edited Daniel, thank you a lot, but I think more changes should be done so I've made changes from my tree and push after test run.

                People

                • Assignee:
                  sanja Oleksandr Byelkin
                  Reporter:
                  ivan.stoykov@skysql.com Stoykov
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  7 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: