db-ojb-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Thomas Mahler <thm...@web.de>
Subject Re: NullPointer in SqlBasedRsIterator issue
Date Thu, 27 Feb 2003 18:01:02 GMT
Hi Per-Olof,

I hope to get it fixed with the information you provided. If I have 
problems I'll contact you.

cheers,
Thomas

Per-Olof Norén wrote:
> Hi Thomas,
> 
> Please let me know if there is need for more info or a patch for my fix.
> / Per-Olof
> 
> Thomas Mahler wrote:
> 
>> Hi Per,
>>
>> I'll have a look at it.
>>
>> Per-Olof Norén wrote:
>>
>>> Hi All,
>>> I think a have found a bug in SqlBasedRsIterator
>>>
>>> Scenario: Using fresch cvs checkout, SingleVM broker:
>>>
>>> When calling:
>>> String query = "select * from a_mapped_table where bla bla";
>>> Collection collection = broker.getCollectionByQuery(query);
>>> One gets:
>>> java.lang.NullPointerException
>>>    at 
>>> org.apache.ojb.broker.accesslayer.RsIterator.getObjectFromResultSet(RsIterator.java:399)

>>>
>>>    at 
>>> org.apache.ojb.broker.accesslayer.SqlBasedRsIterator.getObjectFromResultSet(SqlBasedRsIterator.java:110)

>>>
>>>    at 
>>> org.apache.ojb.broker.accesslayer.RsIterator.next(RsIterator.java:235)
>>>    at 
>>> org.apache.ojb.broker.singlevm.PersistenceBrokerImpl.getCollectionByQuery(PersistenceBrokerImpl.java:1284)

>>>
>>>    at 
>>> org.apache.ojb.broker.singlevm.PersistenceBrokerImpl.getCollectionByQuery(PersistenceBrokerImpl.java:1410)

>>>
>>>    at 
>>> org.apache.ojb.broker.singlevm.PersistenceBrokerImpl.getCollectionByQuery(PersistenceBrokerImpl.java:1456)

>>>
>>>    at 
>>> org.apache.ojb.broker.singlevm.PersistenceBrokerImpl.getCollectionByQuery(PersistenceBrokerImpl.java:1443)

>>>
>>>    at 
>>> org.apache.ojb.broker.singlevm.DelegatingPersistenceBroker.getCollectionByQuery(DelegatingPersistenceBroker.java:310)

>>>
>>>
>>> Which happens because SqlBasedRsIterator never explicitly calls the 
>>> constructor of RsIterator and never assigns the protected cache 
>>> member in rsIterator.
>>> I have copied the snippet of code from RsIterator that assigns the 
>>> cache and placed it in the constructor of SqlBasedRsIterator:
>>> cache = broker.serviceObjectCache();
>>>
>>> This solved seems to have solved the problem.
>>> I can send a patch for this if it´s the correct way of solving the 
>>> troubles.
>>>
>>> ps
>>> I recently sent another patch:
>>> http://archives.apache.org/eyebrowse/ReadMsg?listName=ojb-dev@jakarta.apache.org&msgNo=3688

>>>
>>> on which I haven´t got *any* input on, surely this project is 
>>> accepting any help it can get?
>>
>>
>>
>> Sure! I'll have a look at that fix too.
>>
>> thanks,
>> Thomas
>>
>>> ds
>>>
>>> Regards,
>>> Per-Olof Norén
>>>
> 
> 
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: ojb-dev-unsubscribe@db.apache.org
> For additional commands, e-mail: ojb-dev-help@db.apache.org
> 
> 



Mime
View raw message