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

      Attachments

        Issue Links

          Activity

            Sent for additional testing

            sanja Oleksandr Byelkin added a comment - Sent for additional testing
            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.

            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.
            danblack Daniel Black added a comment - for merging: https://github.com/MariaDB/server/pull/40

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

            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)
            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.

            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

              sanja Oleksandr Byelkin
              ivan.stoykov@skysql.com Stoykov (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              7 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.