1. 01 Nov, 2007 2 commits
  2. 31 Oct, 2007 19 commits
    • unknown's avatar
      16bf418d
    • unknown's avatar
      A fix for Bug#32007 select udf_function() doesn't return an error if error · 189ec0e0
      unknown authored
      during udf initialization. The bug is spotted while working on Bug 12713.
      
      If a user-defined function was used in a SELECT statement, and an
      error would occur during UDF initialization, this error would not terminate
      execution of the SELECT, but rather would be converted to a warning.
      
      The fix is to use a stack buffer to store the message from udf_init instead
      of private my_error() buffer.
      
      
      mysql-test/r/udf.result:
        Update the result to reflect the fix for Bug#32007 select udf_function() 
        doesn't return an error if error during udf initialization
      mysql-test/t/udf.test:
        Update the test to reflect the fix for Bug #32007 select udf_function() 
        doesn't return an error if error during udf initialization
      sql/item_func.cc:
        A fix for Bug#32007.
        
        net.last_error buffer was used to store the temporary message from udf_init.
        Then, when my_error() was called, net.last_error was not empty so 
        my_error() would conclude that there is already an error in the error stack, 
        and not "overwrite" it.
        However, thd->net.report_error was not set, so the the 
        SELECT was not aborted.
        
        The fix is to use a stack buffer instead of thd->net.last_error
        to store the message from udf_init. The message will end up in
        thd->net.last_error anyway after a call to my_error.
      189ec0e0
    • unknown's avatar
      Remove net_printf_error(). Do not talk to network directly in · bfae4730
      unknown authored
      check_user()/check_connection()/check_for_max_user_connections().
      
      This is a pre-requisite patch for the fix for Bug#12713 "Error in a stored 
      function called from a SELECT doesn't cause ROLLBACK of statem"
      
      Implement review comments.
      
      
      sql/mysql_priv.h:
        check_for_max_user_connections() is used in one place only, make it static.
      sql/mysqld.cc:
        Remove net_printf_error(): a consolidation of error reporting facilities
        is necessary to simplify maintenance of the query cache, the 
        client-server protocol, stored procedure continue handlers. 
        Rewrite the only place where its use is somewhat justified 
        (my_error() can not be used since we need to report an error for the thread 
        that does not exist) with my_snprintf()/net_send_error().
      sql/protocol.cc:
        Remove net_printf_error().
      sql/protocol.h:
        Remove net_printf_error().
      sql/sql_connect.cc:
        Remove net_printf_error(). In check_connection()/check_user()/
        check_for_max_user_connections() do not write directly to the network,
        but use the standard my_error() mechanism to record an error in THD.
        It will be sent to the client by the caller. This was the last place
        in the server that would attempt to send an error directly, mainly left 
        untouched by 5.0 refactoring because it is executed only during 
        thread startup.
      sql/sql_parse.cc:
        In the old code, when res was greater than 0, it contained an exact 
        error code, e.g. ER_OUT_OF_RESOURCES or NO SUCH DATABASE, 
        or ER_HANDSHAKE_ERROR. I don't know the reason why this error code was 
        ignored, and instead a generic  ER_UNKNOWN_COM_ERROR was pushed into the
        error stack, but knowing the relaxed attitude towards preserving the error
        codes in the old code, I'm inclinded to think that it was a bug.
        
        After this patch, the most specific error message is already pushed,
        so calling my_message() again is useless.
        
        If res is < 0, the error used to be already sent. This is not done
        by the new code, but will be done later, in the end
        of dispatch_command(). When this is done, clear_error() will be called 
        for us - it is in the first lines of do_command.
        
        To sum up, this change is to remove COM_CHANGE_USER specific error handling
        in favor of the standard one employed for all other COM_* commands.
      bfae4730
    • unknown's avatar
      Merge bk-internal.mysql.com:/home/bk/mysql-5.1-runtime · c2dadab1
      unknown authored
      into  bodhi.(none):/opt/local/work/mysql-5.1-runtime-inc
      
      
      c2dadab1
    • unknown's avatar
      Try to fix a Windows compilation warning. · d757ae80
      unknown authored
      
      sql-common/client.c:
        Move a block of auxiliary functions to the beginning of the file,
        to avoid compilation warnings on Windows.
        Add comments, albeit trivial, while we're at it.
      d757ae80
    • unknown's avatar
      Merge endora.local:/Users/davi/mysql/bugs/31669-5.1 · 17d22305
      unknown authored
      into  endora.local:/Users/davi/mysql/mysql-5.1-runtime
      
      
      17d22305
    • unknown's avatar
      Post merge fix for bug 31669. · 10ed5ae7
      unknown authored
      
      tests/mysql_client_test.c:
        Macro name changed in 5.1
      10ed5ae7
    • unknown's avatar
      Cleanup: rename select_send::status to select_send::is_result_set_started. · 9ad4366b
      unknown authored
      Add select_send::cleanup.
      Fix a compilation warning.
      Issues spotted while working on the fix for Bug#12713.
      
      
      sql-common/client.c:
        Fix a warning.
      sql/sql_class.cc:
        Give a variable a more specific name. Rewrite an incorrect comment.
        Add a cleanup for select_send. The only case now this cleanup can be
        necessary is when we have a prepared statement inside a stored procedure, 
        and a continue handler. At first execution, the statement is killed
        after having executed select_send::send_fields. At the second execution
        it is killed after having executed select_send::send_fields.
      sql/sql_class.h:
        Rename a member. Add comments.
      9ad4366b
    • unknown's avatar
      Merge bk-internal.mysql.com:/home/bk/mysql-5.1-runtime · 6c763974
      unknown authored
      into  bodhi.(none):/opt/local/work/mysql-5.1-runtime-inc
      
      
      libmysql/libmysql.c:
        Auto merged
      6c763974
    • unknown's avatar
      Cleanup: use helper functions to set an error in MYSQL or MYSQL_STMT. · 70987ab9
      unknown authored
      No functionality added or changed.
      This is a pre-requisite for the fix for Bug#12713 Error in a stored 
      function called from a SELECT doesn't cause ROLLBACK of statem
      
      Address post-review comments.
      
      
      include/sql_common.h:
        Declare auxiliary functions to manipulate mysql.net.last_er* and 
        mysql_stmt.last_er*
      libmysql/libmysql.c:
        Use helper functions to set an error in MYSQL or MYSQL_STMT
      libmysqld/lib_sql.cc:
        Use helper functions to set an error in MYSQL or MYSQL_STMT
      sql-common/client.c:
        Use helper functions to set an error in MYSQL or MYSQL_STMT
      70987ab9
    • unknown's avatar
      Merge bk-internal.mysql.com:/home/bk/mysql-5.1-runtime · 44068505
      unknown authored
      into  station.:/mnt/raid/alik/MySQL/devel/5.1-rt-bug31649
      
      
      44068505
    • unknown's avatar
      Merge adventure.(none):/home/thek/Development/cpp/bug31347/my51-bug31347 · ce55bb75
      unknown authored
      into  adventure.(none):/home/thek/Development/cpp/mysql-5.1-runtime
      
      
      sql/sql_acl.cc:
        Auto merged
      ce55bb75
    • unknown's avatar
      Merge adventure.(none):/home/thek/Development/cpp/bug31347/my50-bug31347 · 61c6024c
      unknown authored
      into  adventure.(none):/home/thek/Development/cpp/bug31347/my51-bug31347
      
      
      sql/sql_acl.cc:
        Auto merged
      61c6024c
    • unknown's avatar
      Bug#31347 Increase in memory usage after many DROP USER statements · 8d69dd39
      unknown authored
      Dropping users causes huge increase in memory usage because field values were
      allocated on the server memory root for temporary usage but never deallocated.
      
      This patch changes the target memory root to be that of the thread handler
      instead since this root is cleared between each statement.
      
      
      sql/sql_acl.cc:
        Changed memory root from server life time memory to thread life time memory.
      8d69dd39
    • unknown's avatar
      Merge mysql.com:/home/kent/bk/bug24809/mysql-5.0-build · 41c2a2c4
      unknown authored
      into  mysql.com:/home/kent/bk/bug24809/mysql-5.1-build
      
      
      cmd-line-utils/libedit/Makefile.am:
        Auto merged
      41c2a2c4
    • unknown's avatar
      Fix for a BUG#31649: events.test fails: NULL "state" field of · 47d17e5d
      unknown authored
      SHOW PROCESSLIST.
      
      The problem was a race condition: if the Event Scheduler was not
      quick enough, the following scenario happens:
        - The Event Scheduler picks up the created event;
        - The event is executed;
        - event_scheduler_thread->proc_info is set to NULL;
        - The client issues SELECT FROM I_S.
      
      The fix is to wait for the Event Scheduler to reach 'Waiting
      for next activation' state. 
      
      
      mysql-test/t/events.test:
        Wait for the Event Scheduler to start waiting for the activation.
      47d17e5d
    • unknown's avatar
      Fix result files after patch for BUG#31035. · 77c8c76f
      unknown authored
      
      mysql-test/r/group_min_max.result:
        Update result file.
      mysql-test/r/index_merge_myisam.result:
        Update result file.
      77c8c76f
    • unknown's avatar
      Merge endora.local:/Users/davi/mysql/bugs/31669-5.1 · be4ea151
      unknown authored
      into  endora.local:/Users/davi/mysql/mysql-5.1-runtime
      
      
      be4ea151
    • unknown's avatar
      Merge endora.local:/Users/davi/mysql/mysql-5.0-runtime · 41974cf1
      unknown authored
      into  endora.local:/Users/davi/mysql/bugs/31669-5.1
      
      
      sql/sql_table.cc:
        Auto merged
      libmysql/libmysql.c:
        Manual merge
      tests/mysql_client_test.c:
        Manual merge
      41974cf1
  3. 30 Oct, 2007 16 commits
    • unknown's avatar
      Merge endora.local:/Users/davi/mysql/bugs/30904-5.1 · d2a1fe14
      unknown authored
      into  endora.local:/Users/davi/mysql/mysql-5.1-runtime
      
      
      d2a1fe14
    • unknown's avatar
      Bug#30904 SET PASSWORD statement is non-transactional · bf18f6d4
      unknown authored
      The SET PASSWORD statement is non-transactional (no explicit transaction
      boundaries) in nature and hence is forbidden inside stored functions and
      triggers, but it weren't being effectively forbidden.
      
      The implemented fix is to issue a implicit commit with every SET PASSWORD
      statement, effectively prohibiting these statements in stored functions
      and triggers. 
      
      
      mysql-test/r/sp-error.result:
        Add test case result for Bug#30904
      mysql-test/t/sp-error.test:
        Add test case for Bug#30904
      sql/sql_lex.h:
        Add variable to set that a statement with SET PASSWORD causes a implicit
        commit.
      sql/sql_parse.cc:
        End active transaction in SET PASSWORD.
      sql/sql_yacc.yy:
        Set the correct flag on SET PASSWORD if inside a SP, thus effectively
        prohibiting SET PASSWORD statements in stored functions and triggers.
      bf18f6d4
    • unknown's avatar
      Makefile.am: · 1de7e5c0
      unknown authored
        Ensure use of libedit "config.h" by adding "-I. -I$(srcdir)" to DEFS,
        work around for problem with automake 1.10 (bug#24809)
      
      
      cmd-line-utils/libedit/Makefile.am:
        Ensure use of libedit "config.h" by adding "-I. -I$(srcdir)" to DEFS,
        work around for problem with automake 1.10 (bug#24809)
      1de7e5c0
    • unknown's avatar
      In ha_delete_table, use a standard mechanism to intercept the error message · 8365a74e
      unknown authored
      and convert it to a warning instead of direct manipulation with the
      thread error stack.
      Fix a bug in handler::print_erorr when a garbled message was
      printed for HA_ERR_NO_SUCH_TABLE.
      This is a pre-requisite patch for the fix for Bug#12713 Error in a stored
      function called from a SELECT doesn't cause ROLLBACK of statem
      
      
      sql/handler.cc:
        Use a standard mechanism to intercept the error message, instead
        of direct manipulation with thread error stack. 
        Fix a bug when for HA_ERR_NO_SUCH_TABLE handler::print_error() would
        print a garbled message.
      sql/log.cc:
        Extend internal error handler interface to carry the message text.
      sql/mysqld.cc:
        Extend internal error handler interface to carry the message text.
      sql/sql_base.cc:
        Extend internal error handler interface to carry the message text.
      sql/sql_class.cc:
        Extend internal error handler interface to carry the message text.
      sql/sql_class.h:
        Extend internal error handler interface to carry the message text.
      sql/sql_error.cc:
        Extend internal error handler interface to carry the message text.
      8365a74e
    • unknown's avatar
      Minor Makefile fixes: · 85ba610f
      unknown authored
      1) Add a new top level "Makefile.am" target "make embedded-test".
      
      2) Fix a syntax error (bad merge) in "netware/Makefile.am".
      
      
      Makefile.am:
        Add a new "make" target "embedded-test" so that it is easier 
        to test the "embedded server" ("libmyslqd").
        Make it "phony".
        
        Use it via a sub-make within "test-bt" to avoid code duplication.
      netware/Makefile.am:
        Cleanup: Macros "BUILT_SOURCES" and "CLEANFILES" should be defined only once.
      85ba610f
    • unknown's avatar
      Merge bk-internal.mysql.com:/home/bk/mysql-5.1-runtime · 1430f4de
      unknown authored
      into  bodhi.(none):/opt/local/work/mysql-5.1-runtime-inc-2
      
      
      sql/sql_connect.cc:
        Manual merge.
      1430f4de
    • unknown's avatar
      Use an inline getter method (thd->is_error()) to query if there is an error · 24370012
      unknown authored
      in THD.
      In future the error may be stored elsewhere (not in net.report_error) and 
      it's important to start using an opaque getter to simplify merges.
      
      
      sql/filesort.cc:
        net.report_error -> is_error()
      sql/ha_ndbcluster_binlog.cc:
        net.report_error -> is_error()
      sql/item_func.cc:
        net.report_error -> is_error()
      sql/item_subselect.cc:
        net.report_error -> is_error()
      sql/set_var.cc:
        net.report_error -> is_error()
      sql/sp.cc:
        net.report_error -> is_error()
      sql/sp_head.cc:
        net.report_error -> is_error()
      sql/sql_base.cc:
        net.report_error -> is_error()
      sql/sql_class.cc:
        net.report_error -> is_error()
      sql/sql_class.h:
        net.report_error -> is_error()
      sql/sql_connect.cc:
        net.report_error -> is_error()
      sql/sql_delete.cc:
        net.report_error -> is_error()
      sql/sql_insert.cc:
        net.report_error -> is_error()
      sql/sql_parse.cc:
        net.report_error -> is_error()
      sql/sql_prepare.cc:
        net.report_error -> is_error()
      sql/sql_select.cc:
        net.report_error -> is_error()
      sql/sql_union.cc:
        net.report_error -> is_error()
      sql/sql_update.cc:
        net.report_error -> is_error()
      sql/sql_view.cc:
        net.report_error -> is_error()
      sql/sql_yacc.yy:
        net.report_error -> is_error()
      24370012
    • unknown's avatar
      Merge pippilotta.erinye.com:/shared/home/df/mysql/build/mysql-5.0-build · 4123825d
      unknown authored
      into  pippilotta.erinye.com:/shared/home/df/mysql/build/mysql-5.1-build
      
      
      4123825d
    • unknown's avatar
      Merge dfischer@bk-internal.mysql.com:/home/bk/mysql-5.1-build · 173d8d79
      unknown authored
      into  pippilotta.erinye.com:/shared/home/df/mysql/build/mysql-5.1-build
      
      
      mysql-test/mysql-test-run.pl:
        Auto merged
      173d8d79
    • unknown's avatar
      Merge dfischer@bk-internal.mysql.com:/home/bk/mysql-5.0-build · e3b246d6
      unknown authored
      into  pippilotta.erinye.com:/shared/home/df/mysql/build/mysql-5.0-build
      
      
      e3b246d6
    • unknown's avatar
      Merge pippilotta.erinye.com:/shared/home/df/mysql/build/mysql-5.1-build-work · 2f0838d5
      unknown authored
      into  pippilotta.erinye.com:/shared/home/df/mysql/build/mysql-5.1-build
      
      
      2f0838d5
    • unknown's avatar
      Merge pippilotta.erinye.com:/shared/home/df/mysql/build/mysql-5.0-build-work · 0ca87f8e
      unknown authored
      into  pippilotta.erinye.com:/shared/home/df/mysql/build/mysql-5.1-build-work
      
      
      mysql-test/mysql-test-run.pl:
        Auto merged
      0ca87f8e
    • unknown's avatar
      bug#30630 · b0bc908c
      unknown authored
      b0bc908c
    • unknown's avatar
      Merge moksha.local:/Users/davi/mysql/bugs/31669-5.0 · 157645f4
      unknown authored
      into  moksha.local:/Users/davi/mysql/mysql-5.0-runtime
      
      
      libmysql/libmysql.c:
        Auto merged
      tests/mysql_client_test.c:
        Auto merged
      157645f4
    • unknown's avatar
      Fix failing init_connect.test (5.1-runtime). · d4d4ba9a
      unknown authored
      
      sql/sql_connect.cc:
        Fix failing init_connect.test (5.1-runtime). Add comments for the unjustified use
        of thd->is_slave_error.
      d4d4ba9a
    • unknown's avatar
      Make sure rpl.rpl_innodb_mixed_dml passes even if rpl_mixed.dat is read-only. · 5f4783cf
      unknown authored
      This is important for a development environment where not all source files
      are checked out.
      
      
      5f4783cf
  4. 29 Oct, 2007 3 commits
    • unknown's avatar
      Merge jbruehe@bk-internal.mysql.com:/home/bk/mysql-5.1-build · bd00ccb7
      unknown authored
      into  trift2.:/MySQL/M51/push-5.1
      
      
      bd00ccb7
    • unknown's avatar
      autorun.sh: · 7752cee4
      unknown authored
        After review changes
      
      
      BUILD/autorun.sh:
        After review changes
      7752cee4
    • unknown's avatar
      autorun.sh: · 1b261d3d
      unknown authored
        Removed innobase from compile-dist, and let it use autorun.sh to avoid
        code duplication. Also corrected a problem when searching for one of
        "libtoolize" and "glibtoolize", now it honors the PATH setting (Bug#31405)
      
      
      BUILD/autorun.sh:
        Removed innobase from compile-dist, and let it use autorun.sh to avoid
        code duplication. Also corrected a problem when searching for one of
        "libtoolize" and "glibtoolize", now it honors the PATH setting (Bug#31405)
      1b261d3d