Hi Bathiya, Patch applied ! Thanks providing the patch.
Lahiru On Tue, Aug 14, 2012 at 8:40 AM, Bhathiya Jayasekara <[email protected]> wrote: > Hi Lahiru, > > I attached patch here[1]. > > Thank you. > --Bhathiya > > [1] https://issues.apache.org/jira/browse/AIRAVATA-538 > > > On Mon, Aug 13, 2012 at 8:21 PM, Bhathiya Jayasekara <[email protected]> > wrote: >> >> Hi Lahiru, >> >> I'm happy to do that. I'll submit a patch and let you know. >> >> Thanks. >> --Bhathiya >> >> >> On Mon, Aug 13, 2012 at 8:17 PM, Lahiru Gunathilake <[email protected]> >> wrote: >>> >>> Hi Bathiya, >>> >>> I think its a bug in the code and thank you Bathiya for reporting this >>> issue. YOu are welcome to create a Jira and submit a patch, if you are >>> willing to. Otherwise we can fix it for next release, we like if you >>> can create a jira for this issue. >>> >>> Lahiru >>> >>> On Sun, Aug 12, 2012 at 11:46 AM, Bhathiya Jayasekara >>> <[email protected]> wrote: >>> > Hi devs, >>> > >>> > I think following method in AiravataJCRRegistry is not correct. It >>> > calls >>> > itself recursively. >>> > >>> > public List<WorkflowExecution> getWorkflowExecutionByUser(String user, >>> > >>> > int pageSize, int pageNo) throws RegistryException { >>> > >>> > if (getProvenanceRegistry()!=null){ >>> > >>> > return getProvenanceRegistry().getWorkflowExecutionByUser(user, >>> > pageSize, >>> > pageNo); >>> > >>> > } >>> > return >>> > getWorkflowExecutionByUser(user,pageSize*pageNo,pageSize*(pageNo+1)); >>> > >>> > } >>> > >>> > May be it should call getWorkflowExecution(...) method instead of >>> > itself. Please have a look. >>> > >>> > Thanks. >>> > --Bhathiya >>> >>> >>> >>> -- >>> System Analyst Programmer >>> PTI Lab >>> Indiana University >> >> >> > > > > -- > -Bhathiya Jayasekara- > http://www.google.com/profiles/tobhathiyaj > http://www.facebook.com/#!/bhathiyaj1 > -- System Analyst Programmer PTI Lab Indiana University
