It was just something I'd already added to the commit so I let it ride :-)

shaun

From: Huw Rowlands <notifications@github.com<mailto:notifications@github.com>>
Reply-To: nla/amberdb <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Monday, 17 February 2014 3:23 PM
To: nla/amberdb <amberdb@noreply.github.com<mailto:amberdb@noreply.github.com>>
Cc: Shaun Coen <scoen@nla.gov.au<mailto:scoen@nla.gov.au>>
Subject: Re: [amberdb] Major refactor (#100)


In src/amberdb/sql/BaseGraph.java:

> @@ -25,8 +25,8 @@
>      Map<Object, Edge> graphEdges = new HashMap<Object, Edge>();
>      Map<Object, Vertex> graphVertices = new HashMap<Object, Vertex>();
>
> -public    Map<Object, Set<Edge>> inEdgeSets = new HashMap<Object, Set<Edge>>();
> -public    Map<Object, Set<Edge>> outEdgeSets = new HashMap<Object, Set<Edge>>();
> +    protected Map<Object, Set<Edge>> inEdgeSets = new HashMap<Object, Set<Edge>>();
> +    protected Map<Object, Set<Edge>> outEdgeSets = new HashMap<Object, Set<Edge>>();


Not that this is a huge problem, but did you mean to sneak in this change as well? ;-)

—
Reply to this email directly or view it on GitHub<https://github.com/nla/amberdb/pull/100/files#r9782042>.