WINDUP-139  Create ModelModel and fill the graph.

Ok. In that case I suggest we stick with A for simplicity and clarity.


On Mon, Aug 11, 2014 at 3:03 AM, Ondrej Zizka <notifications@github.com>
wrote:

> In config/impl/src/main/java/org/jboss/windup/config/model/ModelModel.java:
>
> > +package org.jboss.windup.config.model;
> > +
> > +import com.tinkerpop.frames.Property;
> > +import com.tinkerpop.frames.modules.typedgraph.TypeValue;
> > +import org.jboss.windup.graph.model.WindupVertexFrame;
> > +
> > +/**
> > + * Model for Models - to have the information about models in the graph.
> > + * This serves e.g. for reporting.
> > + *
> > + * @author Ondrej Zizka, ozizka at redhat.com
> > + */
> > +@TypeValue("ModelModel")
> > +public interface ModelModel extends WindupVertexFrame
> > +{
> > +    public static final String CLASS = "class";
>
> I have no strong opinion on this:
> A) Normal constants will get mixed with other constants.
> B) Inner enum or interface will create additional code.
> Maybe I would go with B) optionally when some class has more than 5
> properties, or so.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/windup/windup/pull/147/files#r16039615>.
>



-- 
Lincoln Baxter, III
http://ocpsoft.org
"Simpler is better."