Commit 771fb26e authored by aivanov@mysql.com's avatar aivanov@mysql.com

Applied innodb-5.0-ss398 snapshot.

 Fixed BUG#15650: DELETE with LEFT JOIN crashes server
 with innodb_locks_unsafe_for_binlog.
 Fixed compilation problem with non-C99 compilers
 in btr0sea.c
parent 59a30d07
...@@ -59,9 +59,6 @@ before hash index building is started */ ...@@ -59,9 +59,6 @@ before hash index building is started */
#define BTR_SEARCH_BUILD_LIMIT 100 #define BTR_SEARCH_BUILD_LIMIT 100
/* How many cells to check before temporarily releasing btr_search_latch */
#define BTR_CHUNK_SIZE 10000
/************************************************************************ /************************************************************************
Builds a hash index on a page with the given parameters. If the page already Builds a hash index on a page with the given parameters. If the page already
has a hash index with different parameters, the old hash index is removed. has a hash index with different parameters, the old hash index is removed.
...@@ -1607,6 +1604,11 @@ btr_search_validate(void) ...@@ -1607,6 +1604,11 @@ btr_search_validate(void)
mem_heap_t* heap = NULL; mem_heap_t* heap = NULL;
ulint offsets_[REC_OFFS_NORMAL_SIZE]; ulint offsets_[REC_OFFS_NORMAL_SIZE];
ulint* offsets = offsets_; ulint* offsets = offsets_;
/* How many cells to check before temporarily releasing
btr_search_latch. */
ulint chunk_size = 10000;
*offsets_ = (sizeof offsets_) / sizeof *offsets_; *offsets_ = (sizeof offsets_) / sizeof *offsets_;
rw_lock_x_lock(&btr_search_latch); rw_lock_x_lock(&btr_search_latch);
...@@ -1616,7 +1618,7 @@ btr_search_validate(void) ...@@ -1616,7 +1618,7 @@ btr_search_validate(void)
for (i = 0; i < cell_count; i++) { for (i = 0; i < cell_count; i++) {
/* We release btr_search_latch every once in a while to /* We release btr_search_latch every once in a while to
give other queries a chance to run. */ give other queries a chance to run. */
if ((i != 0) && ((i % BTR_CHUNK_SIZE) == 0)) { if ((i != 0) && ((i % chunk_size) == 0)) {
rw_lock_x_unlock(&btr_search_latch); rw_lock_x_unlock(&btr_search_latch);
os_thread_yield(); os_thread_yield();
rw_lock_x_lock(&btr_search_latch); rw_lock_x_lock(&btr_search_latch);
...@@ -1675,8 +1677,8 @@ btr_search_validate(void) ...@@ -1675,8 +1677,8 @@ btr_search_validate(void)
} }
} }
for (i = 0; i < cell_count; i += BTR_CHUNK_SIZE) { for (i = 0; i < cell_count; i += chunk_size) {
ulint end_index = ut_min(i + BTR_CHUNK_SIZE - 1, cell_count - 1); ulint end_index = ut_min(i + chunk_size - 1, cell_count - 1);
/* We release btr_search_latch every once in a while to /* We release btr_search_latch every once in a while to
give other queries a chance to run. */ give other queries a chance to run. */
......
...@@ -989,7 +989,6 @@ innobase_query_caching_of_table_permitted( ...@@ -989,7 +989,6 @@ innobase_query_caching_of_table_permitted(
mutex_enter_noninline(&kernel_mutex); mutex_enter_noninline(&kernel_mutex);
trx_print(stderr, trx, 1024); trx_print(stderr, trx, 1024);
mutex_exit_noninline(&kernel_mutex); mutex_exit_noninline(&kernel_mutex);
ut_error;
} }
innobase_release_stat_resources(trx); innobase_release_stat_resources(trx);
...@@ -3833,6 +3832,13 @@ ha_innobase::unlock_row(void) ...@@ -3833,6 +3832,13 @@ ha_innobase::unlock_row(void)
ut_error; ut_error;
} }
/* Consistent read does not take any locks, thus there is
nothing to unlock. */
if (prebuilt->select_lock_type == LOCK_NONE) {
DBUG_VOID_RETURN;
}
if (srv_locks_unsafe_for_binlog) { if (srv_locks_unsafe_for_binlog) {
row_unlock_for_mysql(prebuilt, FALSE); row_unlock_for_mysql(prebuilt, FALSE);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment