Commit 970fb5fb authored by Rich Prohaska's avatar Rich Prohaska Committed by Yoni Fogel

refs #5763 error write row on auto increment overflow

git-svn-id: file:///svn/mysql/tokudb-engine/tokudb-engine@50963 c7de825b-a66e-492c-adef-691d508d4ae1
parent 153d7f01
...@@ -3804,7 +3804,9 @@ int ha_tokudb::write_row(uchar * record) { ...@@ -3804,7 +3804,9 @@ int ha_tokudb::write_row(uchar * record) {
} }
#endif #endif
if (table->next_number_field && record == table->record[0]) { if (table->next_number_field && record == table->record[0]) {
update_auto_increment(); error = update_auto_increment();
if (error)
goto cleanup;
} }
// //
...@@ -7272,16 +7274,22 @@ void ha_tokudb::init_auto_increment() { ...@@ -7272,16 +7274,22 @@ void ha_tokudb::init_auto_increment() {
void ha_tokudb::get_auto_increment(ulonglong offset, ulonglong increment, ulonglong nb_desired_values, ulonglong * first_value, ulonglong * nb_reserved_values) { void ha_tokudb::get_auto_increment(ulonglong offset, ulonglong increment, ulonglong nb_desired_values, ulonglong * first_value, ulonglong * nb_reserved_values) {
TOKUDB_DBUG_ENTER("ha_tokudb::get_auto_increment"); TOKUDB_DBUG_ENTER("ha_tokudb::get_auto_increment");
ulonglong nr; ulonglong nr;
bool over;
pthread_mutex_lock(&share->mutex); pthread_mutex_lock(&share->mutex);
if (share->auto_inc_create_value > share->last_auto_increment) { if (share->auto_inc_create_value > share->last_auto_increment) {
nr = share->auto_inc_create_value; nr = share->auto_inc_create_value;
over = false;
share->last_auto_increment = share->auto_inc_create_value; share->last_auto_increment = share->auto_inc_create_value;
} }
else { else {
nr = share->last_auto_increment + increment; nr = share->last_auto_increment + increment;
over = nr < share->last_auto_increment;
if (over)
nr = ULONGLONG_MAX;
} }
if (!over) {
share->last_auto_increment = nr + (nb_desired_values - 1)*increment; share->last_auto_increment = nr + (nb_desired_values - 1)*increment;
if (delay_updating_ai_metadata) { if (delay_updating_ai_metadata) {
ai_metadata_update_required = true; ai_metadata_update_required = true;
...@@ -7289,6 +7297,7 @@ void ha_tokudb::get_auto_increment(ulonglong offset, ulonglong increment, ulongl ...@@ -7289,6 +7297,7 @@ void ha_tokudb::get_auto_increment(ulonglong offset, ulonglong increment, ulongl
else { else {
update_max_auto_inc(share->status_block, share->last_auto_increment); update_max_auto_inc(share->status_block, share->last_auto_increment);
} }
}
if (tokudb_debug & TOKUDB_DEBUG_AUTO_INCREMENT) { if (tokudb_debug & TOKUDB_DEBUG_AUTO_INCREMENT) {
TOKUDB_TRACE("get_auto_increment(%lld,%lld,%lld):got:%lld:%lld\n", TOKUDB_TRACE("get_auto_increment(%lld,%lld,%lld):got:%lld:%lld\n",
......
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