1. 04 Jul, 2006 2 commits
    • gkodinov@mysql.com's avatar
      Merge mysql.com:/home/kgeorge/mysql/5.0/teamclean · ae0a3e0d
      gkodinov@mysql.com authored
      into  mysql.com:/home/kgeorge/mysql/5.0/B16110
      ae0a3e0d
    • gkodinov@mysql.com's avatar
      Bug #16110: insert permitted into view col w/o default value · b64089e6
      gkodinov@mysql.com authored
      When compiling INSERT statements the check whether columns are provided values
      depends on the flag whether a field is used in that query (Field::query_id).
      However the check for updatability of VIEW columns (check_view_insertability())
      was calling fix_fields() and thus setting the Field::query_id even for the 
      view fields that are not referenced in the current INSERT statement.
      So the correct check for columns without default values 
      ( check_that_all_fields_are_given_values() ) is assuming that all the VIEW
      columns were mentioned in the INSERT field list and was issuing no 
      warnings or errors.
      Fixed check_view_insertability() to turn off the flag whether or not to set
      Field::query_id (THREAD::set_query_id) before calling fix fields and restore
      it when it's done.
      b64089e6
  2. 29 Jun, 2006 36 commits
  3. 28 Jun, 2006 2 commits
    • kroki@mysql.com's avatar
      Merge mysql.com:/home/tomash/src/mysql_ab/mysql-5.0 · 9c3805c9
      kroki@mysql.com authored
      into  mysql.com:/home/tomash/src/mysql_ab/mysql-5.0-bug10946
      9c3805c9
    • kroki@mysql.com's avatar
      Bug#10946: Confusing error messeges in the case of duplicate trigger definition · 3e2e20ec
      kroki@mysql.com authored
      It was hard to distinguish case, when one was unable to create trigger
      on the table because trigger with same action time and event already
      existed for this table, from the case, when one tried to create trigger
      with name which was already occupied by some other trigger, since in
      both these cases we emitted ER_TRG_ALREADY_EXISTS error and message.
      Now we emit ER_NOT_SUPPORTED_YET error with appropriate additional
      message in the first case. There is no sense in introducing separate
      error for this situation since we plan to get rid of this limitation
      eventually.
      3e2e20ec