Code review comment for lp:~zorba-coders/zorba/structuralrelationships2

Revision history for this message
Federico Cavalieri (fcavalieri) wrote :

> Federico, I don't think it is necessary to put the position uris into the uri
> pool because they are not likely to be shared by multiple data items so you
> won't really be saving much space (you may be actually end-up wasting space
> because the uri pool storage has some overhead by itself). Do you agree?

I agree.
Fixing it.

Thanks
Federico

« Back to merge proposal