[rules-users] Drools Flow: Variable Persistence Strategies
Salaboy
salaboy at gmail.com
Fri Feb 19 19:12:53 EST 2010
Yes that's a common practice! Sorry for the delay in my answer. I will
try to test and your contribution asap!
Greetings!
- Ing. Mauricio Salatino -
On Feb 19, 2010, at 15:59, Dale Wyttenbach <dale.wyttenbach at gmail.com>
wrote:
> Here's another idea to bounce off of you: Given
> StringVariablePersister, you can easily persist things other than
> Strings, so long as you can construct them from a String. (Boolean,
> Integer, etc). In the attached patch, I abstracted your
> StringVariablePersister and created a BooleanVariablePersister.
> What do you think?
>
> 2010/2/18 Mauricio Salatino <salaboy at gmail.com>
> Yes, of course, I add the VPS feature into drools so, I'm glad to
> see that it's useful, I will add that persister, and if you are
> interested in do some contributions with JCR will be nice too!
> Thanks a lot for your contributions!
>
>
> 2010/2/18 Dale Wyttenbach <dale.wyttenbach at gmail.com>
> Excellent, I have attached a unit test that demonstrates a query
> made possible by the first patch, in case you are interested. The
> unit test is designed to run in your example DroolsFlowVPS project,
> since StringVariablePersister is not part of drools-persistence-
> jpa. (Could StringVariablePersister be contributed to drools?)
>
> 2010/2/18 Mauricio Salatino <salaboy at gmail.com>
> The first patch is already in the trunk.
> About the second problem, we are trying to remove the
> @CollectionOfElements annotation, (because it's hibernate specific)
> that's why you see the other one commented out. I will try fix it
> asap, to be able to use toplink as persistence provider as well.
> Probably that will fix your problem too.
>
>
>
> 2010/2/18 Dale Wyttenbach <dale.wyttenbach at gmail.com>
> That's correct, it works fine using h2. Thanks.
>
> 2010/2/18 Mauricio Salatino <salaboy at gmail.com>
>
> Let me take a look at that too.. probably it's an oracle problem
> right? I mean, probably it just happen with oracle.. we can create a
> fix for that too
>
>
> 2010/2/18 Dale Wyttenbach <dale.wyttenbach at gmail.com>
>
> My pleasure.
>
> Within ProcessInstanceInfo, I've got a similar observation about the
> eventTypes member. 'private Set<EventType>' is currently commented
> out in favor of 'private Set<String>'. When using an Oracle
> hibernate dialect, this results in the DDL 'create table
> ProcessInstanceInfo_eventTypes' which yields the error 'ORA-00972:
> identifier is too long'. I imagine this will be resolved when the
> String is promoted to EventType but I thought you might want to know.
>
> 2010/2/13 Mauricio Salatino <salaboy at gmail.com>
>
> done.. I will apply the patch into the trunk.. it seems to be correct.
> Thanks a lot!
>
> 2010/2/13 Dale Wyttenbach <dale.wyttenbach at gmail.com>
>
> Please see attached.
>
>
> 2010/2/12 Mauricio Salatino <salaboy at gmail.com>
> I just want to make sure before apply the patch that the change will
> not impact in other places.
> Thanks
>
>
> On Fri, Feb 12, 2010 at 5:38 PM, Mauricio Salatino
> <salaboy at gmail.com> wrote:
> can you post both, previous and new, sql schema generated for the
> VariableInstanceInfo and ProcessInstanceInfo tables?
>
>
>
> 2010/2/12 Dale Wyttenbach <dale.wyttenbach at gmail.com>
> If you try to write a query that requires
> VariableInstanceInfo.processId, it won't work because that class has
> no processId member.
> I guessed that processInstanceId would be a better join column, and
> that worked (patch attached)
>
> Thanks for the reply.
>
> 2010/2/12 Mauricio Salatino <salaboy at gmail.com>
>
> Hmm..
> I'm looking at the source and I see the following mapping:
>
> @OneToMany(cascade = CascadeType.ALL)
> @JoinColumn(name = "processId")
> @MapKey(name = "name")
> private Map<String, VariableInstanceInfo> variables =
> new HashMap<String, VariableInstanceInfo>();
>
> That makes me think that there is a join column called processId
> between VariableInstanceInfo and ProcessInstanceInfo.
>
>
> 2010/2/12 Dale Wyttenbach <dale.wyttenbach at gmail.com>
> I have a patch for drools-persistence-jpa should anyone agree with
> me that ProcessInstanceInfo and VariableInstanceInfo should be
> joined by processInstanceId.
>
>
> On Fri, Feb 12, 2010 at 10:52 AM, Dale Wyttenbach <dale.wyttenbach at gmail.com
> > wrote:
> I'm trying to adapt the StringVariablePersister presented in http://blog.athico.com/2009/09/drools-flow-variable-persistence.html
>
> I can create process instances with variable "color":
>
> private static WorkflowProcessInstance startProcess
> (StatefulKnowledgeSession ksession, String color) {
> Map<String, Object> vars = new HashMap<String, Object>();
> vars.put("color", color);
> WorkflowProcessInstance p1 = (WorkflowProcessInstance)
> ksession.startProcess("com.sample.ruleflow", vars);
> return p1;
> }
>
> I can count them:
>
> select
> count(*)
> from
> ProcessInstanceInfo
>
> However I'm stymied as to counting process instances with a given
> color, as there appears to be no join column between
> ProcessInstanceInfo and VariableInstanceInfo.
> Can such a query be written?
>
> Thanks.
>
>
>
>
>
> _______________________________________________
> rules-users mailing list
> rules-users at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/rules-users
>
>
>
>
> --
> - http://salaboy.wordpress.com
> - http://www.jbug.com.ar
> - Salatino "Salaboy" Mauricio -
>
> _______________________________________________
> rules-users mailing list
> rules-users at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/rules-users
>
>
>
> _______________________________________________
> rules-users mailing list
> rules-users at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/rules-users
>
>
>
>
> --
> - http://salaboy.wordpress.com
> - http://www.jbug.com.ar
> - Salatino "Salaboy" Mauricio -
>
>
>
> --
> - http://salaboy.wordpress.com
> - http://www.jbug.com.ar
> - Salatino "Salaboy" Mauricio -
>
> _______________________________________________
> rules-users mailing list
> rules-users at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/rules-users
>
>
>
> _______________________________________________
> rules-users mailing list
> rules-users at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/rules-users
>
>
>
>
> --
> - http://salaboy.wordpress.com
> - http://www.jbug.com.ar
> - Salatino "Salaboy" Mauricio -
>
> _______________________________________________
> rules-users mailing list
> rules-users at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/rules-users
>
>
>
> _______________________________________________
> rules-users mailing list
> rules-users at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/rules-users
>
>
>
>
> --
> - http://salaboy.wordpress.com
> - http://www.jbug.com.ar
> - Salatino "Salaboy" Mauricio -
>
> _______________________________________________
> rules-users mailing list
> rules-users at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/rules-users
>
>
>
> _______________________________________________
> rules-users mailing list
> rules-users at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/rules-users
>
>
>
>
> --
> - http://salaboy.wordpress.com
> - http://www.jbug.com.ar
> - Salatino "Salaboy" Mauricio -
>
> _______________________________________________
> rules-users mailing list
> rules-users at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/rules-users
>
>
>
> _______________________________________________
> rules-users mailing list
> rules-users at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/rules-users
>
>
>
>
> --
> - http://salaboy.wordpress.com
> - http://www.jbug.com.ar
> - Salatino "Salaboy" Mauricio -
>
> _______________________________________________
> rules-users mailing list
> rules-users at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/rules-users
>
>
> <patch.txt.gz>
> _______________________________________________
> rules-users mailing list
> rules-users at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/rules-users
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/rules-users/attachments/20100219/849022d2/attachment.html
More information about the rules-users
mailing list