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(a)gmail.com&gt; wrote: > I'm trying to adapt the StringVariablePersister presented in > http://blog.athico.com/2009/09/drools-flow-variable-persistence.html > > <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(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/rules-users

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(a)gmail.com&gt; 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(a)gmail.com&gt; > >> 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(a)gmail.com&gt; wrote: >> >>> I'm trying to adapt the StringVariablePersister presented in >>> http://blog.athico.com/2009/09/drools-flow-variable-persistence.html >>> >>> <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(a)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(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/rules-users > > _______________________________________________ rules-users mailing list rules-users(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/rules-users

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(a)gmail.com&gt;wrote: > can you post both, previous and new, sql schema generated for the > VariableInstanceInfo and ProcessInstanceInfo tables? > > > > 2010/2/12 Dale Wyttenbach <dale.wyttenbach(a)gmail.com&gt; > >> 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(a)gmail.com&gt; >> >> 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(a)gmail.com&gt; >>> >>>> 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(a)gmail.com&gt; wrote: >>>> >>>>> I'm trying to adapt the StringVariablePersister presented in >>>>> http://blog.athico.com/2009/09/drools-flow-variable-persistence.html >>>>> >>>>> <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(a)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(a)lists.jboss.org >>> https://lists.jboss.org/mailman/listinfo/rules-users >>> >>> >> >> _______________________________________________ >> rules-users mailing list >> rules-users(a)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(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/rules-users

done.. I will apply the patch into the trunk.. it seems to be correct. Thanks a lot! 2010/2/13 Dale Wyttenbach <dale.wyttenbach(a)gmail.com&gt; Please see attached. > > > 2010/2/12 Mauricio Salatino <salaboy(a)gmail.com&gt; > >> 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(a)gmail.com&gt;wrote: >> >>> can you post both, previous and new, sql schema generated for the >>> VariableInstanceInfo and ProcessInstanceInfo tables? >>> >>> >>> >>> 2010/2/12 Dale Wyttenbach <dale.wyttenbach(a)gmail.com&gt; >>> >>>> 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(a)gmail.com&gt; >>>> >>>> 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(a)gmail.com&gt; >>>>> >>>>>> 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(a)gmail.com&gt; wrote: >>>>>> >>>>>>> I'm trying to adapt the StringVariablePersister presented in >>>>>>> http://blog.athico.com/2009/09/drools-flow-variable-persistence.html >>>>>>> >>>>>>> <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(a)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(a)lists.jboss.org >>>>> https://lists.jboss.org/mailman/listinfo/rules-users >>>>> >>>>> >>>> >>>> _______________________________________________ >>>> rules-users mailing list >>>> rules-users(a)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(a)lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/rules-users >> >> > > _______________________________________________ > rules-users mailing list > rules-users(a)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(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/rules-users

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(a)gmail.com&gt; 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(a)gmail.com&gt; > > done.. I will apply the patch into the trunk.. it seems to be correct. >> Thanks a lot! >> >> 2010/2/13 Dale Wyttenbach <dale.wyttenbach(a)gmail.com&gt; >> >> Please see attached. >>> >>> >>> 2010/2/12 Mauricio Salatino <salaboy(a)gmail.com&gt; >>> >>>> 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(a)gmail.com&gt;wrote: >>>> >>>>> can you post both, previous and new, sql schema generated for the >>>>> VariableInstanceInfo and ProcessInstanceInfo tables? >>>>> >>>>> >>>>> >>>>> 2010/2/12 Dale Wyttenbach <dale.wyttenbach(a)gmail.com&gt; >>>>> >>>>>> 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(a)gmail.com&gt; >>>>>> >>>>>> 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(a)gmail.com&gt; >>>>>>> >>>>>>>> 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(a)gmail.com&gt; wrote: >>>>>>>> >>>>>>>>> I'm trying to adapt the StringVariablePersister presented in >>>>>>>>> http://blog.athico.com/2009/09/drools-flow-variable-persistence.html >>>>>>>>> >>>>>>>>> <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(a)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(a)lists.jboss.org >>>>>>> https://lists.jboss.org/mailman/listinfo/rules-users >>>>>>> >>>>>>> >>>>>> >>>>>> _______________________________________________ >>>>>> rules-users mailing list >>>>>> rules-users(a)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(a)lists.jboss.org >>>> https://lists.jboss.org/mailman/listinfo/rules-users >>>> >>>> >>> >>> _______________________________________________ >>> rules-users mailing list >>> rules-users(a)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(a)lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/rules-users >> >> > > _______________________________________________ > rules-users mailing list > rules-users(a)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(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/rules-users

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(a)gmail.com&gt; > That's correct, it works fine using h2. Thanks. > > 2010/2/18 Mauricio Salatino <salaboy(a)gmail.com&gt; > > 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(a)gmail.com&gt; >> >> 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(a)gmail.com&gt; >>> >>> done.. I will apply the patch into the trunk.. it seems to be correct. >>>> Thanks a lot! >>>> >>>> 2010/2/13 Dale Wyttenbach <dale.wyttenbach(a)gmail.com&gt; >>>> >>>> Please see attached. >>>>> >>>>> >>>>> 2010/2/12 Mauricio Salatino <salaboy(a)gmail.com&gt; >>>>> >>>>>> 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(a)gmail.com&gt; wrote: >>>>>> >>>>>>> can you post both, previous and new, sql schema generated for the >>>>>>> VariableInstanceInfo and ProcessInstanceInfo tables? >>>>>>> >>>>>>> >>>>>>> >>>>>>> 2010/2/12 Dale Wyttenbach <dale.wyttenbach(a)gmail.com&gt; >>>>>>> >>>>>>>> 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(a)gmail.com&gt; >>>>>>>> >>>>>>>> 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(a)gmail.com&gt; >>>>>>>>> >>>>>>>>>> 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(a)gmail.com&gt; wrote: >>>>>>>>>> >>>>>>>>>>> I'm trying to adapt the StringVariablePersister presented in >>>>>>>>>>> http://blog.athico.com/2009/09/drools-flow-variable-persistence.html >>>>>>>>>>> >>>>>>>>>>> <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(a)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(a)lists.jboss.org >>>>>>>>> https://lists.jboss.org/mailman/listinfo/rules-users >>>>>>>>> >>>>>>>>> >>>>>>>> >>>>>>>> _______________________________________________ >>>>>>>> rules-users mailing list >>>>>>>> rules-users(a)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(a)lists.jboss.org >>>>>> https://lists.jboss.org/mailman/listinfo/rules-users >>>>>> >>>>>> >>>>> >>>>> _______________________________________________ >>>>> rules-users mailing list >>>>> rules-users(a)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(a)lists.jboss.org >>>> https://lists.jboss.org/mailman/listinfo/rules-users >>>> >>>> >>> >>> _______________________________________________ >>> rules-users mailing list >>> rules-users(a)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(a)lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/rules-users >> >> > > _______________________________________________ > rules-users mailing list > rules-users(a)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(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/rules-users

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(a)gmail.com&gt; > 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(a)gmail.com&gt; > >> 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(a)gmail.com&gt; >> >>> That's correct, it works fine using h2. Thanks. >>> >>> 2010/2/18 Mauricio Salatino <salaboy(a)gmail.com&gt; >>> >>> 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(a)gmail.com&gt; >>>> >>>> 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(a)gmail.com&gt; >>>>> >>>>> done.. I will apply the patch into the trunk.. it seems to be correct. >>>>>> Thanks a lot! >>>>>> >>>>>> 2010/2/13 Dale Wyttenbach <dale.wyttenbach(a)gmail.com&gt; >>>>>> >>>>>> Please see attached. >>>>>>> >>>>>>> >>>>>>> 2010/2/12 Mauricio Salatino <salaboy(a)gmail.com&gt; >>>>>>> >>>>>>>> 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(a)gmail.com&gt; wrote: >>>>>>>> >>>>>>>>> can you post both, previous and new, sql schema generated for the >>>>>>>>> VariableInstanceInfo and ProcessInstanceInfo tables? >>>>>>>>> >>>>>>>>> >>>>>>>>> >>>>>>>>> 2010/2/12 Dale Wyttenbach <dale.wyttenbach(a)gmail.com&gt; >>>>>>>>> >>>>>>>>>> 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(a)gmail.com&gt; >>>>>>>>>> >>>>>>>>>> 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(a)gmail.com&gt; >>>>>>>>>>> >>>>>>>>>>>> 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(a)gmail.com&gt; wrote: >>>>>>>>>>>> >>>>>>>>>>>>> I'm trying to adapt the StringVariablePersister presented in >>>>>>>>>>>>> http://blog.athico.com/2009/09/drools-flow-variable-persistence.html >>>>>>>>>>>>> >>>>>>>>>>>>> <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(a)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(a)lists.jboss.org >>>>>>>>>>> https://lists.jboss.org/mailman/listinfo/rules-users >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>> >>>>>>>>>> _______________________________________________ >>>>>>>>>> rules-users mailing list >>>>>>>>>> rules-users(a)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(a)lists.jboss.org >>>>>>>> https://lists.jboss.org/mailman/listinfo/rules-users >>>>>>>> >>>>>>>> >>>>>>> >>>>>>> _______________________________________________ >>>>>>> rules-users mailing list >>>>>>> rules-users(a)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(a)lists.jboss.org >>>>>> https://lists.jboss.org/mailman/listinfo/rules-users >>>>>> >>>>>> >>>>> >>>>> _______________________________________________ >>>>> rules-users mailing list >>>>> rules-users(a)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(a)lists.jboss.org >>>> https://lists.jboss.org/mailman/listinfo/rules-users >>>> >>>> >>> >>> _______________________________________________ >>> rules-users mailing list >>> rules-users(a)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(a)lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/rules-users >> >> > > _______________________________________________ > rules-users mailing list > rules-users(a)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(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/rules-users