Mailing-List: contact dev-help@deltacloud.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@deltacloud.apache.org
Received-SPF: pass (athena.apache.org: domain of mfojtik@redhat.com designates
 209.132.183.28 as permitted sender)
Date: Thu, 10 May 2012 10:25:25 +0200
From: Michal Fojtik <mfojtik@redhat.com>
To: dev@deltacloud.apache.org
Subject: Re: [PATCH core 03/32] Core: Added wrapper for autoloading
 collections
Message-ID: <20120510082525.GA93411@dhcp-29-121.redhat.com>
References: <1334670011-40658-1-git-send-email-mfojtik@redhat.com>
 <1334670011-40658-3-git-send-email-mfojtik@redhat.com>
 <1336609085.2686.25.camel@avon.watzmann.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1336609085.2686.25.camel@avon.watzmann.net>
User-Agent: Mutt/1.5.21 (2011-07-01)

On 05/10/12, David Lutterkort wrote:
> On Tue, 2012-04-17 at 15:39 +0200, mfojtik@redhat.com wrote:
> > From: Michal Fojtik <mfojtik@redhat.com>
> 
> [ I am looking at your branch, but it's more convenient to reply to
> individual patches ]
> 
> This and 2/32 seem like a little too much magic to me; why do this
> rather than have a big server.rb ? It's 1200 lines right now, but they
> are fairly boring code, and I like being able to see all that code at
> once.

Sorry, I was too fast with replying ;-) You are referring to the collection
subfolder not to lazy-loading.

My thoughts was to make it more readable and structured when it will be
splitted into separate files.

Having everything in one place is nice but when you first look into the
server.rb it honestly looks like a mess :-)

IMHO having the collections in separate files will make it more easy to fix
things in particular collection, without making people rebasing and fixing
conflicts in one big server.rb.

Also since this code use Base class from Sinatra, all collections are just
'modules' that can be plugged and un-plugged as needed. So you can for
example use DC as base for your application where you choose just
collections you really want to use.

Another thing is that I guess in future we will need to add more
collections, like for managing networks/etc so the amount of code in
server.rb will eventually grow. I think that for the people that want to
add new collection to Deltacloud, it's now more easy just 'copy' the
collection file, rename it and add their stuff there.

  -- Michal

-- 
Michal Fojtik
Sr. Software Engineer, Deltacloud API (http://deltacloud.org)