From dev-return-9724-apmail-openjpa-dev-archive=openjpa.apache.org@openjpa.apache.org Fri Dec 05 15:24:07 2008 Return-Path: Delivered-To: apmail-openjpa-dev-archive@www.apache.org Received: (qmail 7043 invoked from network); 5 Dec 2008 15:24:03 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 5 Dec 2008 15:24:03 -0000 Received: (qmail 92241 invoked by uid 500); 5 Dec 2008 15:24:15 -0000 Delivered-To: apmail-openjpa-dev-archive@openjpa.apache.org Received: (qmail 92215 invoked by uid 500); 5 Dec 2008 15:24:15 -0000 Mailing-List: contact dev-help@openjpa.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@openjpa.apache.org Delivered-To: mailing list dev@openjpa.apache.org Received: (qmail 92203 invoked by uid 99); 5 Dec 2008 15:24:14 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 05 Dec 2008 07:24:14 -0800 X-ASF-Spam-Status: No, hits=2.4 required=10.0 tests=HTML_MESSAGE,SPF_PASS,WHOIS_MYPRIVREG X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of allee8285@gmail.com designates 209.85.217.19 as permitted sender) Received: from [209.85.217.19] (HELO mail-gx0-f19.google.com) (209.85.217.19) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 05 Dec 2008 15:22:44 +0000 Received: by gxk12 with SMTP id 12so59372gxk.9 for ; Fri, 05 Dec 2008 07:23:25 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:message-id:date:from:to :subject:in-reply-to:mime-version:content-type:references; bh=a10/NEPm+6sJK/mKgu2nN0Y5lPedPGp4tv4Cr2BUn2I=; b=nP0DPug9jb0fknHIZvWX/BorHYHkxN9BZCEC3nwKZL5f0RqR8cVAxOPlyJxMYC8gCv jwgRHyRqTkmVXDdTxxxsi1W4UwYPVMqPUVV2varLTghvreNssGAhWYAojGICsE2OMPry oltxxeJJgpMN8mL1MZSWIs4olE4SmhJw+QgaQ= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:to:subject:in-reply-to:mime-version :content-type:references; b=b8VVZpTuuYIWn/X/MWvcdL8wNEX5VSExokNdwjuwzkx7tyvBN5URaEug9PFPNxLN0x W8ylqmmI4QIFCLBjC4ylaqy1MH2nUQLn+HKpj7YLtZbV5xi0TvkDnsXgD9p3kaFyscby B1iGC90FeNnetwbTiG2/HOgD8pmmQTsu8FAbw= Received: by 10.142.177.7 with SMTP id z7mr45979wfe.59.1228490600357; Fri, 05 Dec 2008 07:23:20 -0800 (PST) Received: by 10.142.102.3 with HTTP; Fri, 5 Dec 2008 07:23:20 -0800 (PST) Message-ID: <8e68c8e90812050723j24479997n3c617560639596e6@mail.gmail.com> Date: Fri, 5 Dec 2008 09:23:20 -0600 From: "Albert Lee" To: dev@openjpa.apache.org, mikedd@apache.org Subject: Re: [VOTE] Turn off enhancement by subclassing as the default In-Reply-To: <72c1350f0812041929t396393cdq3ba606c39ba1c45f@mail.gmail.com> MIME-Version: 1.0 Content-Type: multipart/alternative; boundary="----=_Part_33667_24102139.1228490600302" References: <89c0c52c0812041625x69489674x59d052ffae528e37@mail.gmail.com> <72c1350f0812041929t396393cdq3ba606c39ba1c45f@mail.gmail.com> X-Virus-Checked: Checked by ClamAV on apache.org ------=_Part_33667_24102139.1228490600302 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline Just want to point out that there is no impact for application deployed in app server which will perform automatic class enhancement even no build time enhancement or javaagent is specified. Albert Lee. On Thu, Dec 4, 2008 at 9:29 PM, Michael Dick wrote: > Hi David, > > You're correct on both counts. You'll only be affected if you do not run > the > PCEnhancer, or use a javaagent. > > My Votes : > +1 Turn off subclass enhancement in trunk > > +1 Turn off subclass enhancement in 1.3.x > In for a penny in for a pound. . . > > I think the subclass support a noble goal but it needs some additional work > before it should be the default behavior. Until we find a champion who is > interested and has the time to dedicate to ironing out the bugs we should > leave it turned off. > > -mike > > On Thu, Dec 4, 2008 at 7:51 PM, David Jencks > wrote: > > > Just to be really clear, this won't affect either: > > > > - projects that have pre-enhanced their classes at build time using e.g. > a > > maven plugin or ant task > > - app servers (for instance) that start up with an appropriate javaagent > > that hooks up to the openjpa enhancer > > > > it will only affect people who don't make any effort to enhance their > > classes > > > > Correct? > > > > thanks > > david jencks > > > > > > On Dec 4, 2008, at 4:25 PM, Kevin Sutter wrote: > > > > Hi, > >> This is a tough decision, but one that I think we need to make. If you > >> have > >> been following the dev mailing list, there have been several discussions > >> [1] > >> and JIRA Issues [2] about the fallback enhancement by subclassing that > we > >> put in place back in the 1.0.0 timeframe. Although we succeeded in > making > >> the initial out-of-box experience easier for the newbie OpenJPA > developer, > >> we also masked the need for true enhancement for production usage. So, > >> unless we deem that this subclassing enhancement is critical to > OpenJPA's > >> acceptance and usage, I propose to turn this option off by default. The > >> ability to do this subclass enhancement will still be available via the > >> openjpa.RuntimeUnenhancedClasses property, but the default will now be > >> either "warn" or "unsupported" instead of "supported". I would like to > do > >> this for trunk for sure and possibly the 1.3.x branch as well. Please > >> vote > >> accordingly. Thanks for your input. Write-in comments are also > welcome. > >> > >> [ +1 | 0 | -1 ] Turn off subclass enhancement in trunk > >> [ +1 | 0 | -1 ] Turn off subclass enhancement in 1.3.x > >> > >> I am not proposing to turn it off in the other branches since those are > >> not > >> active development streams, but rather service streams. We shouldn't > >> introduce a change like this into a customer's service stream. That is, > >> for > >> a customer to get 1.0.4 with this option turned off would be a surprise > >> since they would only be expecting fixes. Fine line in this case, but > you > >> get the picture. > >> > >> Thanks, > >> Kevin > >> > >> [1] > >> > >> > http://n2.nabble.com/Re%3A-Foreign-key-field-doesn%27t-get-populated-in-descendant-class-in-Join-Inheritance-td1574111.html#a1574493 > >> [2] http://issues.apache.org/jira/browse/OPENJPA-651, > >> http://issues.apache.org/jira/browse/OPENJPA-650, > >> https://issues.apache.org/jira/browse/OPENJPA-293 > >> > > > > > -- Albert Lee. ------=_Part_33667_24102139.1228490600302--