Changeset: ac82e696762a for MonetDB
URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=ac82e696762a
Modified Files:
monetdb5/modules/mal/remote.mx
Branch: default
Log Message:
remote: allow get to retrieve OID-headed columns too
An OID-headed dense BAT is easy enough like a VOID-headed one. Allow to
send such BAT. Sloppily allow both VOID and OID-headed bats from
remote.get, since we can't know the target BAT is dense or not there.
Would basically require a fallback method instead of a-priori choice.
diffs (21 lines):
diff --git a/monetdb5/modules/mal/remote.mx b/monetdb5/modules/mal/remote.mx
--- a/monetdb5/modules/mal/remote.mx
+++ b/monetdb5/modules/mal/remote.mx
@@ -649,7 +649,7 @@
rt, ident);
GDKfree(rt);
- if (isaBatType(rtype) && (localtype == 0 || localtype != c->type ||
getHeadType(rtype) != TYPE_void))
+ if (isaBatType(rtype) && (localtype == 0 || localtype != c->type ||
(getHeadType(rtype) != TYPE_void && getHeadType(rtype) != TYPE_oid)))
{
int h, t, s;
ptr l, r;
@@ -1208,7 +1208,7 @@
if (b == NULL)
throw(MAL, "remote.bincopyto", RUNTIME_OBJECT_UNDEFINED);
- if (b->htype != TYPE_void)
+ if (!(b->htype == TYPE_void || (b->htype == TYPE_oid && b->T->dense)))
throw(ILLARG, "remote.bincopyto", "only void-headed BATs are
supported");
mnstr_printf(cntxt->fdout, /*JSON*/"{"
_______________________________________________
Checkin-list mailing list
[email protected]
http://mail.monetdb.org/mailman/listinfo/checkin-list
