Commit ca9dc9c2 authored by Zardosht Kasheff's avatar Zardosht Kasheff Committed by Yoni Fogel

[t:3996], use bulk fetch in add_index

git-svn-id: file:///svn/mysql/tokudb-engine/tokudb-engine@35797 c7de825b-a66e-492c-adef-691d508d4ae1
parent 7698f060
...@@ -7092,6 +7092,8 @@ int ha_tokudb::tokudb_add_index( ...@@ -7092,6 +7092,8 @@ int ha_tokudb::tokudb_add_index(
bool rw_lock_taken = false; bool rw_lock_taken = false;
*inc_num_DBs = false; *inc_num_DBs = false;
*modified_DBs = false; *modified_DBs = false;
invalidate_bulk_fetch();
unpack_entire_row = true; // for bulk fetching rows
for (u_int32_t i = 0; i < MAX_KEY+1; i++) { for (u_int32_t i = 0; i < MAX_KEY+1; i++) {
mult_put_flags[i] = 0; mult_put_flags[i] = 0;
mult_dbt_flags[i] = DB_DBT_REALLOC; mult_dbt_flags[i] = DB_DBT_REALLOC;
...@@ -7218,6 +7220,14 @@ int ha_tokudb::tokudb_add_index( ...@@ -7218,6 +7220,14 @@ int ha_tokudb::tokudb_add_index(
else { else {
rw_unlock(&share->num_DBs_lock); rw_unlock(&share->num_DBs_lock);
rw_lock_taken = false; rw_lock_taken = false;
struct smart_dbt_bf_info bf_info;
bf_info.ha = this;
// you need the val if you have a clustering index and key_read is not 0;
bf_info.direction = 1;
bf_info.thd = ha_thd();
bf_info.need_val = TRUE;
error = db_env->create_loader( error = db_env->create_loader(
db_env, db_env,
txn, txn,
...@@ -7257,13 +7267,38 @@ int ha_tokudb::tokudb_add_index( ...@@ -7257,13 +7267,38 @@ int ha_tokudb::tokudb_add_index(
); );
if (error) { goto cleanup; } if (error) { goto cleanup; }
cursor_ret_val = tmp_cursor->c_get(tmp_cursor, &curr_pk_key, &curr_pk_val, DB_NEXT | DB_PRELOCKED); cursor_ret_val = tmp_cursor->c_getf_next(tmp_cursor, DB_PRELOCKED,smart_dbt_bf_callback, &bf_info);
while (cursor_ret_val != DB_NOTFOUND) { while (cursor_ret_val != DB_NOTFOUND || ((bytes_used_in_range_query_buff - curr_range_query_buff_offset) > 0)) {
if (cursor_ret_val) { if ((bytes_used_in_range_query_buff - curr_range_query_buff_offset) == 0) {
invalidate_bulk_fetch();
cursor_ret_val = tmp_cursor->c_getf_next(tmp_cursor, DB_PRELOCKED, smart_dbt_bf_callback, &bf_info);
if (cursor_ret_val != DB_NOTFOUND && cursor_ret_val != 0) {
error = cursor_ret_val; error = cursor_ret_val;
goto cleanup; goto cleanup;
} }
}
if ((bytes_used_in_range_query_buff - curr_range_query_buff_offset) == 0) {
break;
}
// get key info
uchar* curr_pos = range_query_buff+curr_range_query_buff_offset;
u_int32_t key_size = *(u_int32_t *)curr_pos;
curr_pos += sizeof(key_size);
uchar* curr_key_buff = curr_pos;
curr_pos += key_size;
curr_pk_key.data = curr_key_buff;
curr_pk_key.size = key_size;
u_int32_t val_size = *(u_int32_t *)curr_pos;
curr_pos += sizeof(val_size);
uchar* curr_val_buff = curr_pos;
curr_pos += val_size;
curr_pk_val.data = curr_val_buff;
curr_pk_val.size = val_size;
curr_range_query_buff_offset = curr_pos - range_query_buff;
error = loader->put(loader, &curr_pk_key, &curr_pk_val); error = loader->put(loader, &curr_pk_key, &curr_pk_val);
if (error) { goto cleanup; } if (error) { goto cleanup; }
...@@ -7283,7 +7318,6 @@ int ha_tokudb::tokudb_add_index( ...@@ -7283,7 +7318,6 @@ int ha_tokudb::tokudb_add_index(
goto cleanup; goto cleanup;
} }
} }
cursor_ret_val = tmp_cursor->c_get(tmp_cursor, &curr_pk_key, &curr_pk_val, DB_NEXT | DB_PRELOCKED);
} }
error = tmp_cursor->c_close(tmp_cursor); error = tmp_cursor->c_close(tmp_cursor);
assert(error==0); assert(error==0);
......
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