MonetDB: new_rmt_opt - Improves comments, adds assert

[stefanos mavros via checkin-list]

Changeset: 56151b3a90a2 for MonetDB
URL: https://dev.monetdb.org/hg/MonetDB/rev/56151b3a90a2
Modified Files:
        sql/server/rel_distribute.c
Branch: new_rmt_opt
Log Message:

Improves comments, adds assert


diffs (31 lines):

diff --git a/sql/server/rel_distribute.c b/sql/server/rel_distribute.c
--- a/sql/server/rel_distribute.c
+++ b/sql/server/rel_distribute.c
@@ -271,6 +271,7 @@ rel_rewrite_remote_(visitor *v, sql_rel 
                                tu->uri = mapiuri_uri(t->query, v->sql->sa);
                                list *uris = sa_list(v->sql->sa);
                                append(uris, tu);
+
                                p = rel->p = prop_create(v->sql->sa, 
PROP_REMOTE, rel->p);
                                p->id = 0;
                                p->value.pval = (void *)uris;
@@ -279,8 +280,8 @@ rel_rewrite_remote_(visitor *v, sql_rel 
                if (t && isReplicaTable(t) && !list_empty(t->members)) {
                        /* the parts of a replica are either
                         * - remote tables for which we have to store tid and 
uri
-                        * - local table for which we only care if they exist
-                        * the relevant info are passed in
+                        * - local table for which we only care if they exist 
(localpart var)
+                        * the relevant info are passed in the REMOTE property 
value.pval and id members
                         */
                        list *uris = sa_list(v->sql->sa);
                        sqlid localpart = 0;
@@ -302,6 +303,8 @@ rel_rewrite_remote_(visitor *v, sql_rel 
                                p = rel->p = prop_create(v->sql->sa, 
PROP_REMOTE, rel->p);
                                p->id = localpart;
                                p->value.pval = (void*)uris;
+                       } else {
+                               assert(localpart);
                        }
                }
        } break;
_______________________________________________
checkin-list mailing list -- checkin-list@monetdb.org
To unsubscribe send an email to checkin-list-le...@monetdb.org