[jbossseam-issues] [JBoss JIRA] Closed: (JBSEAM-1635) Migrating to Drools 4.0.0 MR3

[Norman Richards (JIRA)]

     [ http://jira.jboss.com/jira/browse/JBSEAM-1635?page=all ]

Norman Richards closed JBSEAM-1635.
-----------------------------------

    Resolution: Done

I've upgraded drools.   There was one tricky problem: this version of drools added something called shadow facts.  This is implemented by a proxy.  The problem with the proxy is that  is incompatible with the javassist proxies used in Seam due to the use of final methods by javassist proxies.  (also a problem with cglib proxies)   Shadow proxies can be disabled in drools, but this results in drools giving wrong answers when facts change. 

I've had to duplicate the javassist ProxyFactory in Seam, modified to not generate any final methods.  This should not affect anything else.  In theory, it could slightly decrease performance my limiting optimization possibilities.  It also could also introduce javassist version dependencies should the underlying javassist jar change drastically.  The only other option would appear to be never asserting any proxied Seam components into drools, so it seems like the best choice if we want support future versions of drools.




> Migrating to Drools 4.0.0 MR3
> -----------------------------
>
>                 Key: JBSEAM-1635
>                 URL: http://jira.jboss.com/jira/browse/JBSEAM-1635
>             Project: JBoss Seam
>          Issue Type: Feature Request
>          Components: Drools
>    Affects Versions: 2.0.0.BETA1
>         Environment: OS: all. JBoss AS 4.2.0 GA. Seam CVS 2007-07-10, Drools SVN 2007-07-10
>            Reporter: Ning Zhao
>         Assigned To: Norman Richards
>             Fix For: 2.0.0.CR1
>
>         Attachments: DroolsHandler.java, RuleBase.java, RuleBasedIdentity.java
>
>
> Drools API has been changed since 4.0.0M3. Please see:
> http://wiki.jboss.org/wiki/Wiki.jsp?page=4.0.0.MR3ReleaseNotes
> Corresponding changes in the Seam codebase and Seam reference should be made if Drools 4.0 GA is to be used in the future Seam. 

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira