[JENA-116] DatasetGraphMap.getGraph() should call addGraph() instead of adding directly to graph collection

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Done
• Fix Version/s: Jena 2.11.0
• Component/s: ARQ
• Assignee: Andy Seaborne
• Reporter: Stephen Allen
• Created: 14/Sep/11 19:27
• Updated: 01/Feb/15 19:12
• Resolved: 01/Feb/15 19:12

Description

The DatasetGraphMap class is useful for implementing a quad store as a collection of triplestores, but when you attempt to retrieve a graph that does not exist, a new one is created but bypasses the addGraph() method.

Parliament needs to register all new graphs in an internal table (so that it knows which directory and files on disk correspond to each graph). It does this by overriding the addGraph() method.

I've attached a patch that fixes this issue and also removes the "final" modifier on the getGraph() method (not strictly necessary for our use case, but it seemed out of place since no other methods are final, and I can imagine implementations that may want to override that method).