Return-Path: Delivered-To: apmail-db-ojb-dev-archive@www.apache.org Received: (qmail 94067 invoked from network); 5 Jul 2007 15:48:10 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 5 Jul 2007 15:48:10 -0000 Received: (qmail 39995 invoked by uid 500); 5 Jul 2007 15:48:13 -0000 Delivered-To: apmail-db-ojb-dev-archive@db.apache.org Received: (qmail 39974 invoked by uid 500); 5 Jul 2007 15:48:13 -0000 Mailing-List: contact ojb-dev-help@db.apache.org; run by ezmlm Precedence: bulk List-Unsubscribe: List-Help: List-Post: List-Id: "OJB Developers List" Reply-To: "OJB Developers List" Delivered-To: mailing list ojb-dev@db.apache.org Received: (qmail 39963 invoked by uid 99); 5 Jul 2007 15:48:12 -0000 Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 05 Jul 2007 08:48:12 -0700 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests= X-Spam-Check-By: apache.org Received-SPF: neutral (herse.apache.org: local policy) Received: from [213.115.255.60] (HELO manta.curalia.se) (213.115.255.60) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 05 Jul 2007 08:48:06 -0700 Received: from manta.curalia.se (manta.curalia.se [213.115.149.212]) by manta.curalia.se (Postfix) with ESMTP id 165DEABC006 for ; Thu, 5 Jul 2007 17:47:44 +0200 (CEST) Received: by manta.curalia.se (Postfix, from userid 65534) id 0AA0EABC008; Thu, 5 Jul 2007 17:47:44 +0200 (CEST) Received: from [192.168.4.63] (soekris.curalia.se [213.115.149.209]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by manta.curalia.se (Postfix) with ESMTP id 6C4BBABC006 for ; Thu, 5 Jul 2007 17:47:42 +0200 (CEST) Message-ID: <468D129E.8090500@apache.org> Date: Thu, 05 Jul 2007 17:47:42 +0200 From: =?UTF-8?B?TWFydGluIEthbMOpbg==?= Organization: ASF User-Agent: Thunderbird 1.5.0.12 (X11/20070509) MIME-Version: 1.0 To: OJB Developers List Subject: Re: svn commit: r538346 - in /db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker: PersistenceChecker.java core/PersistenceCheckerOptImpl.java core/PersistenceCheckerSolidImpl.java util/ShortcutMapper.java References: <20070515224132.E272E1A9838@eris.apache.org> In-Reply-To: <20070515224132.E272E1A9838@eris.apache.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Checker-Version: SpamAssassin 3.0.3 (2005-04-27) on manta.curalia.se X-Spam-Level: X-AV-Checked: ClamAV using ClamSMTP X-Virus-Checked: Checked by ClamAV on apache.org X-Old-Spam-Status: No, score=-2.1 required=5.0 tests=ALL_TRUSTED,AWL autolearn=ham version=3.0.3 Hi Armin and everyone else, snippets from SVN #538346: arminw@apache.org wrote: > URL: http://svn.apache.org/viewvc?view=rev&rev=538346 > Log: > initial check in > > Added: > db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/util/ShortcutMapper.java > > + static > + { > + Mapping mapping = new Mapping(ObjectCache.class) > + .add("jcs", ObjectCacheJCSImpl.class); This adds a new run-time dependency for OJB (the JCS JAR) which was previously only present if explicitly selecting the JCS cache. Would it be possible to change to use class names as String objects stored in the shortcut mapping table, and then look them up on demand instead (with the OJB standard classForName wrappers)? I could work on a patch for this if you think it seems like an OK behviour. (The change would be transparent to users having the shortcut "jcs" in their repository files.) I think it's nice to have the number of OJB runtime deps small and add 3rd party JARs only when required (DBCP is another one that I think should be reworked like this, ie only needed at RT when explicitly selecting DBCP pooling). Any thoughts? Regards, Martin --------------------------------------------------------------------- To unsubscribe, e-mail: ojb-dev-unsubscribe@db.apache.org For additional commands, e-mail: ojb-dev-help@db.apache.org