Github user ajs6f commented on the pull request:
https://github.com/apache/jena/pull/95#issuecomment-212405267
Another +1 to @osma's comments. In re: serialization: it seems to me that
the need here is for "replay-ability". Certainly serialization will do that,
but at a very high cost, and the cost will vary with client behavior (suppose
everyone asks for RDF/XML vs. everyone asks for Turtle). There are cheaper ways
to get "rewindable-ity", such as have been mentioned (`ResultSetRewindable`).
Recently, @afs did work with node caching that showed impressive impact. If the
results aren't serialized here, maybe that same technique could be used to
further decrease memory consumption.
On another point, I suspect that immutable `CacheEntry` might be easier to
reason about going forward, but maybe that's just me.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
