continuum-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Carlos Sanchez" <car...@apache.org>
Subject Re: svn commit: r449533 [1/3] - in /maven/continuum/trunk: ./ continuum-api/src/main/java/org/apache/maven/continuum/ continuum-api/src/main/java/org/apache/maven/continuum/project/builder/ continuum-api/src/main/java/org/apache/maven/continuum/store
Date Tue, 26 Sep 2006 23:45:22 GMT
Thanks forthe check, my eyes were hurting after i finished with this ;)

On 9/26/06, Brett Porter <brett@apache.org> wrote:
> I've taken a peek through this to sanity check - there are some
> things that look like they've been removed that shouldn't have been,
> but I could be wrong and it could have been intentional. Can you
> confirm?
>
> On 25/09/2006, at 10:40 AM, carlos@apache.org wrote:
>
> > Added:
> >     maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/
> > maven/continuum/web/filter/
> >       - copied from r448283, maven/continuum/branches/continuum-
> > acegi/continuum-webapp/src/main/java/org/apache/maven/continuum/web/
> > filter/
> >     maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/
> > maven/continuum/web/filter/FilterToComponentProxy.java
> >       - copied unchanged from r448283, maven/continuum/branches/
> > continuum-acegi/continuum-webapp/src/main/java/org/apache/maven/
> > continuum/web/filter/FilterToComponentProxy.java
>
> Isn't this only for Acegi?


For now yes. It's general purpose and should be moved to plexus


>
> > Removed:
> >     maven/continuum/trunk/continuum-webapp/src/main/webapp/error.jsp
>
> Isn't this needed?

moved to error/error.jsp

>
> > -    ProjectGroup getDefaultProjectGroup()
> > -        throws ContinuumStoreException;
> > -
>
> was this intentional?

where is this?

>
> > Modified: maven/continuum/trunk/continuum-core/src/main/java/org/
> > apache/maven/continuum/DefaultContinuum.java
> >
> > -        try
> > -        {
> > -            buildResults = new ArrayList
> > ( store.getProjectWithBuilds( projectId ).getBuildResults() );
> > -        }
> > -        catch ( ContinuumObjectNotFoundException e )
> > -        {
> > -            return Collections.EMPTY_LIST;
> > -        }
> > -        catch ( ContinuumStoreException e )
> > -        {
> > -            throw logAndCreateException( "Exception while getting
> > build results for project.", e );
> > -        }
> > +        buildResults = new ArrayList
> > ( store.getBuildResultsForProject( projectId, 0 ) );
>
> was this intentional? If so, I assume all the other exception
> removals were too so that there is a catch all. I'm still not sure I
> like that, but something we can overall review later.

persistence layer exceptions should not be catched if not doing
anything, so they can be handled in upper levels or just cause an
internal error.
For instance a ObjectNotFound could mean to the web layer that the
object was deleted while the user was reading the page.

>
> > Modified: maven/continuum/trunk/continuum-webapp/src/main/filters/
> > filter.properties
> > URL: http://svn.apache.org/viewvc/maven/continuum/trunk/continuum-
> > webapp/src/main/filters/filter.properties?
> > view=diff&rev=449533&r1=449532&r2=449533
> > ======================================================================
> > ========
> > --- maven/continuum/trunk/continuum-webapp/src/main/filters/
> > filter.properties (original)
> > +++ maven/continuum/trunk/continuum-webapp/src/main/filters/
> > filter.properties Sun Sep 24 17:40:23 2006
> > @@ -2,3 +2,37 @@
> >  # emails to the specified address
> >  # nobody@localhost
> >  mail.override.to.address=
> > +
> > +#
> > +# Database configuration
> > +#
> > +
> > +# Derby
> > +db.driverName=org.apache.derby.jdbc.EmbeddedDriver
> > +db.url=jdbc:derby:${plexus.home}/database;create=true
> > +db.userName=sa
> > +db.password=
> > +acl.sql=org/apache/maven/user/acegi/acl/acegi-acl-derby.sql
> > +
> > +# PostgreSQL
> > +#db.driverName=org.postgresql.Driver
> > +#db.url=jdbc:postgresql://localhost/continuum
> > +#db.userName=
> > +#db.password=
> > +#acl.sql=not done yet
> > +
> > +# MySQL
> > +#db.driverName=com.mysql.jdbc.Driver
> > +#db.url=jdbc:mysql://localhost/test
> > +#db.userName=
> > +#db.password=
> > +#acl.sql=org/apache/maven/user/acegi/acl/acegi-acl-mysql.sql
> > +
> > +# HSQLDB
> > +# NOTE: NO NOT USE THIS CONFIGURATION FOR A PRODUCTION SYSTEM.
> > +# HSQLDB keeps all data in memory at all times.
> > +#db.driverName=org.hsqldb.jdbcDriver
> > +#db.url=jdbc:hsqldb:${plexus.home}/database
> > +#db.userName=sa
> > +#db.password=
> > +#acl.sql=not done yet
>
> Can we change this to the datasource based one, consistent with
> Archiva? Much nicer to externalise configuration from the webapp for
> distribution purposes.


Yes, that should be done, still I like the filter so I can use a
different db for development avoiding conficts in application.xml


>
> - Brett
>


-- 
I could give you my word as a Spaniard.
No good. I've known too many Spaniards.
                             -- The Princess Bride

Mime
View raw message