[ 
https://issues.apache.org/jira/browse/SYNAPSE-533?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13995761#comment-13995761
 ] 

Hiranya Jayathilaka commented on SYNAPSE-533:
---------------------------------------------

Hi Max,

I had a quick look and the code seems to be correct. However, you might want to 
think about the case where the select query returns a very large number of 
rows. You might want to add some optional mechanisms to prevent an out of 
memory error in such situations. Other than that, it looks good to me.

-Hiranya

> DBLookup extracs only first element from database
> -------------------------------------------------
>
>                 Key: SYNAPSE-533
>                 URL: https://issues.apache.org/jira/browse/SYNAPSE-533
>             Project: Synapse
>          Issue Type: Improvement
>          Components: Extension Mediators, Script mediator
>    Affects Versions: 1.2
>            Reporter: minal sontakke
>            Assignee: Hiranya Jayathilaka
>             Fix For: 2.1
>
>         Attachments: SYNAPSE-533.patch
>
>   Original Estimate: 96h
>  Remaining Estimate: 96h
>
> I just realized that the DBLookup mediator extracts only the first row... :( 
> when i try to use statement like select * from mytable;  and if my table 
> called as mytable contains 3 values like test1, test2 and test 3, when i try 
> to use the same only first element gets displayed ie like test1 and i dont 
> get output as test1,test2, test3.  DBLookup needs to improve this to extract 
> multiple rows if there are multiple rows, in which case the result set will 
> be a list of results which might be a special list that needs to be implement 
> because it can get a XML representation of this list easily.
> Refer discussion at link : http://wso2.org/forum/thread/5181



--
This message was sent by Atlassian JIRA
(v6.2#6252)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to