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
