MonetDB: Dec2023 - UNLOGGED tables do have to be logged when the...

[Sjoerd Mullender via checkin-list]

Changeset: 1cc64810035e for MonetDB
URL: https://dev.monetdb.org/hg/MonetDB/rev/1cc64810035e
Modified Files:
        sql/storage/bat/bat_storage.c
Branch: Dec2023
Log Message:

UNLOGGED tables do have to be logged when they are dropped.
Fixes #7517.


diffs (30 lines):

diff --git a/sql/storage/bat/bat_storage.c b/sql/storage/bat/bat_storage.c
--- a/sql/storage/bat/bat_storage.c
+++ b/sql/storage/bat/bat_storage.c
@@ -3734,7 +3734,7 @@ drop_del(sql_trans *tr, sql_table *t)
 
        if (!isNew(t)) {
                storage *bat = ATOMIC_PTR_GET(&t->data);
-               trans_add_obj(tr, &t->base, bat, &tc_gc_del, 
&commit_destroy_del, NOT_TO_BE_LOGGED(t) ? NULL : &log_destroy_del);
+               trans_add_obj(tr, &t->base, bat, &tc_gc_del, 
&commit_destroy_del, isTempTable(t) ? NULL : &log_destroy_del);
        }
        return ok;
 }
@@ -3744,7 +3744,7 @@ drop_col(sql_trans *tr, sql_column *c)
 {
        assert(!isNew(c));
        sql_delta *d = ATOMIC_PTR_GET(&c->data);
-       trans_add(tr, &c->base, d, &tc_gc_col, &commit_destroy_del, 
NOT_TO_BE_LOGGED(c->t) ? NULL : &log_destroy_col);
+       trans_add(tr, &c->base, d, &tc_gc_col, &commit_destroy_del, 
isTempTable(c->t) ? NULL : &log_destroy_col);
        return LOG_OK;
 }
 
@@ -3753,7 +3753,7 @@ drop_idx(sql_trans *tr, sql_idx *i)
 {
        assert(!isNew(i));
        sql_delta *d = ATOMIC_PTR_GET(&i->data);
-       trans_add(tr, &i->base, d, &tc_gc_idx, &commit_destroy_del, 
NOT_TO_BE_LOGGED(i->t) ? NULL : &log_destroy_idx);
+       trans_add(tr, &i->base, d, &tc_gc_idx, &commit_destroy_del, 
isTempTable(i->t) ? NULL : &log_destroy_idx);
        return LOG_OK;
 }
 
_______________________________________________
checkin-list mailing list -- checkin-list@monetdb.org
To unsubscribe send an email to checkin-list-le...@monetdb.org