Changeset: 0241383ebc0d for MonetDB
URL: https://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=0241383ebc0d
Modified Files:
        sql/storage/store.c
Branch: Nov2019
Log Message:

Non regular tables might get renamed or get their access changed on a 
transaction, so allow them in the rollforward_update_table code


diffs (33 lines):

diff --git a/sql/storage/store.c b/sql/storage/store.c
--- a/sql/storage/store.c
+++ b/sql/storage/store.c
@@ -3787,16 +3787,19 @@ rollforward_update_table(sql_trans *tr, 
                        if (bs_debug) 
                                fprintf(stderr, "#update table %s\n", 
tt->base.name);
                        ok = store_funcs.update_table(tr, ft, tt);
-                       ft->cleared = 0;
-                       tt->access = ft->access;
-
-                       if (ok == LOG_OK && isRenamed(ft)) { /* apply possible 
renaming */
-                               list_hash_delete(tt->s->tables.set, tt, NULL);
-                               tt->base.name = sa_strdup(tr->parent->sa, 
ft->base.name);
-                               if (!list_hash_add(tt->s->tables.set, tt, NULL))
-                                       ok = LOG_ERR;
-                               setRenamedFlag(tt); /* propagate the change to 
the upper transaction */
-                       }
+               }
+       }
+
+       if (mode == R_APPLY && ok == LOG_OK) {
+               ft->cleared = 0;
+               tt->access = ft->access;
+
+               if (isRenamed(ft)) { /* apply possible renaming */
+                       list_hash_delete(tt->s->tables.set, tt, NULL);
+                       tt->base.name = sa_strdup(tr->parent->sa, 
ft->base.name);
+                       if (!list_hash_add(tt->s->tables.set, tt, NULL))
+                               ok = LOG_ERR;
+                       setRenamedFlag(tt); /* propagate the change to the 
upper transaction */
                }
        }
        return ok;
_______________________________________________
checkin-list mailing list
[email protected]
https://www.monetdb.org/mailman/listinfo/checkin-list

Reply via email to