Re: [chain2] configuration façade APIs

2012-07-27 Thread Oliver Heger
i To: Commons Developers List Sent: Thursday, 26 July 2012 4:58 AM Subject: Re: [chain2] configuration façade APIs I may draft up a prototype using YAML as a configuration source, just to make sure that it in fact is a good abstraction. I noticed that the SnakeYaml parser is under the Apache 2.0 l

Re: [chain2] configuration façade APIs - followup

2012-07-27 Thread Simone Tripodi
Hi Elijah, thanks a lot for the prompt feedback! I will start just moving the modules (don't think that this deserves an issue) than will fill issues for new tasks - and feel free to submit code!!! Thanks a lot! -Simo http://people.apache.org/~simonetripodi/ http://simonetripodi.livejournal.com/

Re: [chain2] configuration façade APIs

2012-07-27 Thread Elijah Zupancic
Bruno P. Kinoshita >>> >> wrote: >>> >>> >>> >>> +1 >>> >>> >>> >>> I used SnakeYaml in a project [1] that parses TAP test streams and in >>> >>> some Jenkins plug-ins, and had a look at the sourc

Re: [chain2] configuration façade APIs - followup

2012-07-27 Thread Elijah Zupancic
Simo, I love ALL of those ideas. It took me quite a bit of time to get to where we are now and I considered it a first draft that we would then refactor from. So, yes please! -Elijah On Fri, Jul 27, 2012 at 4:16 AM, Simone Tripodi wrote: > Hi Elijah/all, > > I just checked out latest modificati

[chain2] configuration façade APIs - followup

2012-07-27 Thread Simone Tripodi
Hi Elijah/all, I just checked out latest modifications to see the changes - well done and thanks for leading that! - I have few questions: * the main façade is a little "obscure" to me - how the parser returns the Catalog/Chain to Parser clients? Would it be useful adding a method (or modify the

Re: [chain2] configuration façade APIs

2012-07-27 Thread Simone Tripodi
1] that parses TAP test streams and in >> >>> some Jenkins plug-ins, and had a look at the source code too. It works >> very >> >>> well with the latest YAML spec and the source code is very neat and >> with >> >>> many tests. >> >>> >> >>> TestNG uses SnakeYaml f

Re: [chain2] configuration façade APIs

2012-07-26 Thread Elijah Zupancic
YAML spec and the source code is very neat and > with > >>> many tests. > >>> > >>> TestNG uses SnakeYaml for parsing YAML configuration of test suites too > >>> [2]. > >>> > >>> [1] http://www.tap4j.org > >>>

Re: [chain2] configuration façade APIs

2012-07-26 Thread Simone Tripodi
gt;> >>> [1] http://www.tap4j.org >>> [2] https://github.com/cbeust/testng/blob/master/pom.xml#L124 >>> >>> Bruno P. Kinoshita >>> http://kinoshita.eti.br >>> http://tupilabs.com >>> >>>> __

Re: [chain2] configuration façade APIs

2012-07-26 Thread Oliver Heger
Bruno P. Kinoshita http://kinoshita.eti.br http://tupilabs.com From: Simone Tripodi To: Commons Developers List Sent: Thursday, 26 July 2012 4:58 AM Subject: Re: [chain2] configuration façade APIs I may draft up a prototype using YAML as a configuration

Re: [chain2] configuration façade APIs

2012-07-26 Thread Simone Tripodi
t;To: Commons Developers List >>Sent: Thursday, 26 July 2012 4:58 AM >>Subject: Re: [chain2] configuration façade APIs >> >>> I may draft up a prototype using YAML as a configuration source, just >>> to make sure that it in fact is a good abstraction. I noticed

Re: [chain2] configuration façade APIs

2012-07-26 Thread Bruno P. Kinoshita
2012 4:58 AM >Subject: Re: [chain2] configuration façade APIs > >> I may draft up a prototype using YAML as a configuration source, just >> to make sure that it in fact is a good abstraction. I noticed that the >> SnakeYaml parser is under the Apache 2.0 license >>

Re: [chain2] configuration façade APIs

2012-07-26 Thread Simone Tripodi
> I may draft up a prototype using YAML as a configuration source, just > to make sure that it in fact is a good abstraction. I noticed that the > SnakeYaml parser is under the Apache 2.0 license > (http://code.google.com/p/snakeyaml/). I'm assuming that it wouldn't > be a problem to take it as a d

Re: [chain2] configuration façade APIs

2012-07-25 Thread Elijah Zupancic
I may draft up a prototype using YAML as a configuration source, just to make sure that it in fact is a good abstraction. I noticed that the SnakeYaml parser is under the Apache 2.0 license (http://code.google.com/p/snakeyaml/). I'm assuming that it wouldn't be a problem to take it as a dependency.

Re: [chain2] configuration façade APIs

2012-07-25 Thread Elijah Zupancic
Hi everyone, I've created a first draft of refactoring chain so that it uses a facade for configuration. Please see the diff attached to this ticket to get my proposed changes: https://issues.apache.org/jira/browse/CHAIN-72 Here is a summary of what I have done: * Removed the module configurati

Re: [chain2] configuration façade APIs

2012-07-24 Thread Oliver Heger
Hi Simo and Elijah, Am 23.07.2012 21:55, schrieb Simone Tripodi: Thanks a lot Oliver, much more than appreciated! If you could have a look at current configuration stuff at [chain2] and share what you think would be already *great*! I had a look at the current config module. I understand Elij

Re: [chain2] configuration façade APIs

2012-07-23 Thread Simone Tripodi
Thanks a lot Oliver, much more than appreciated! If you could have a look at current configuration stuff at [chain2] and share what you think would be already *great*! then, feel free to put your hands and help us on defining the façade :) alles gute, -Simo http://people.apache.org/~simonetripo

Re: [chain2] configuration façade APIs

2012-07-23 Thread Simone Tripodi
Hi Elijah, not sure we understand each other there, please confirm IIUC - in my mind I would have extracted "configurations APIs" (mainly interfaces and few abstract classes maybe) from the configuration module and put in the core, then renamed the actual configuration module to `xml-configuration

Re: [chain2] configuration façade APIs

2012-07-23 Thread Oliver Heger
Am 23.07.2012 09:00, schrieb Simone Tripodi: Good morning all, so I continue proposing the already proposed roadmap: let's add the façade APIs for the [chain] configuration stuff, adapt the existing XML configuration reader, use the [configuration] in future releases for new [chain] configuratio

Re: [chain2] configuration façade APIs

2012-07-23 Thread Elijah Zupancic
I've been going through the chain source for about a day looking for a way to decouple the digester configuration. Sadly, I don't think that I will be able to do it without removing digester. I may just jump ahead and remove [digester] completely and then move us to [configuration] for creating dyn

Re: [chain2] configuration façade APIs

2012-07-23 Thread Elijah Zupancic
Hi Simo, That sounds good to me. However, I'm having trouble separating the existing Catalog implementation from the rest of chain. Digester is tightly coupled across components, so it is a non-trivial refactor to make a configuration facade. I'm working on it, but it will take some time. Thanks,

Re: [chain2] configuration façade APIs

2012-07-23 Thread Simone Tripodi
Good morning all, so I continue proposing the already proposed roadmap: let's add the façade APIs for the [chain] configuration stuff, adapt the existing XML configuration reader, use the [configuration] in future releases for new [chain] configurations. How does it sound? best, -Simo http://peo

Re: [chain2] configuration façade APIs

2012-07-22 Thread Elijah Zupancic
Hi Oliver, Configuration seems like it might be useful if I end up redoing the XML configuration portion. Are there any plans to support YAML? Thanks, -Elijah On Sun, Jul 22, 2012 at 1:16 PM, Oliver Heger wrote: > Hi Simo, > > Am 22.07.2012 17:54, schrieb Simone Tripodi: > >> Good point Oliver,

Re: [chain2] configuration façade APIs

2012-07-22 Thread Oliver Heger
Hi Simo, Am 22.07.2012 17:54, schrieb Simone Tripodi: Good point Oliver, I honestly didn't think about [configuration], please apologize! since [chain] already had a way to be configured via an XML wrapper around the Digester, we thought it would have been good having a façade and plug other te

Re: [chain2] configuration façade APIs

2012-07-22 Thread Simone Tripodi
Good point Oliver, I honestly didn't think about [configuration], please apologize! since [chain] already had a way to be configured via an XML wrapper around the Digester, we thought it would have been good having a façade and plug other textual format... Anyway, we are open to suggestions - wha

Re: [chain2] configuration façade APIs

2012-07-22 Thread Oliver Heger
Is there any relation or overlap to [configuration]? Depending on your concrete requirements, this is probably over-sized. But maybe a source of inspiration? Oliver Am 22.07.2012 10:00, schrieb Elijah Zupancic: Thanks! On Sat, Jul 21, 2012 at 11:54 PM, Simone Tripodi wrote: Just tracked C

Re: [chain2] configuration façade APIs

2012-07-22 Thread Elijah Zupancic
Thanks! On Sat, Jul 21, 2012 at 11:54 PM, Simone Tripodi wrote: > Just tracked CHAIN-72 and assigned to Elijah > > best, > -Simo > > http://people.apache.org/~simonetripodi/ > http://simonetripodi.livejournal.com/ > http://twitter.com/simonetripodi > http://www.99soft.org/ > > > On Sat, Jul 21, 2

Re: [chain2] configuration façade APIs

2012-07-21 Thread Simone Tripodi
Just tracked CHAIN-72 and assigned to Elijah best, -Simo http://people.apache.org/~simonetripodi/ http://simonetripodi.livejournal.com/ http://twitter.com/simonetripodi http://www.99soft.org/ On Sat, Jul 21, 2012 at 11:27 PM, Simone Tripodi wrote: > Hi all guys, > > Elijah and and I had a chat

[chain2] configuration façade APIs

2012-07-21 Thread Simone Tripodi
Hi all guys, Elijah and and I had a chat and we thought that, since chain2 hasn't released yet, we are still in time to define a façade API for textual configurations and rename the current XML configuration module to xml-configuration (and adapt it to new API) - new formats such as YAML and JSON