geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Matt Hogstrom (JIRA)" <>
Subject [jira] Commented: (DAYTRADER-9) MarketSummary fix for NPE with less than 106 quotes
Date Thu, 27 Jul 2006 02:10:14 GMT
    [ ] 
Matt Hogstrom commented on DAYTRADER-9:


there is an if in the patch

+            if ( (topGainersData.size() > 0) || (topGainersData.size() > 0)){

I'm not sure why the comparison is duplicated.  Did you mean to have another value for the
comparison?  I expect we only need one of the comparisons but wanted to check.

> MarketSummary fix for NPE with less than 106 quotes
> ---------------------------------------------------
>                 Key: DAYTRADER-9
>                 URL:
>             Project: DayTrader
>          Issue Type: Improvement
>          Components: EJB Tier
>    Affects Versions: 1.1
>         Environment: All OS / all platforms
>            Reporter: Slava
>         Attachments: DAYTRADER-9.patch
> This is a fix for the NPE in the MarketSummary.jsp and TradeHome.jsp thrown when the
Daytrader database contains only 100 or less quotes. (see
> The current default configuration is 200 users and 400 quotes.  As previously discussed,
it takes long time to populate Daytrader database with as many user/quotes.
> This fix solves that problem.  No NPE will be thrown if the database is populated with
less than 105 quotes.  This will speed up significantly the functional test with Daytrader.
> However, to retrieve market summary data you still need to have more than 106 quotes
in the database.  Therefore for performance measurments with Daytrader should be done using
 higher number of users/quotes.
> Slava

This message is automatically generated by JIRA.
If you think it was sent incorrectly contact one of the administrators:
For more information on JIRA, see:


View raw message