Hi, Tikuna!

Why don't you create a Jira issue of type improvement and submit a pull 
request so that the NH team can evaluate it?
You should consider the following:

- Does it work in all/most supported database servers?
- Does it break anything?
- Is the solution generic enough, or does it only work under certain 
scenarios?

Anyway, it's always good to have people interested in moving NH forward! ;-)

RP


On Tuesday, July 14, 2015 at 12:26:14 PM UTC+1, tikuna wrote:
>
> Hello everyone!
>
> Did any of you had this problem before?
>
> Any of you have any concern about why will not be a good idea to make a 
> change to fix this?
>
> We have a patch for this and will be happy to share it. It's  good to take 
> 5 minutes to check it out since gives a big pump in performance.
>
> Thanks!
>
> Best Regard!
>
> El lunes, 30 de marzo de 2015, 14:34:32 (UTC-3), tikuna escribió:
>>
>> Hi,
>>
>> We are using NHibernate, and got some processing that modify a big chunk 
>> of data; and are using Version control for Optimistic Locking.
>>
>> In the process, inserts are done using batch processing with very good 
>> performance, but updates are doing a data base roundtrip each (if I take 
>> out optimistic locking with version from the entity, updates are executed 
>> in batch's).
>>
>> I find in the documentation this:
>>
>>
>>    - 
>>    
>>    optimistic concurrency checking may be impaired since ADO.NET 2.0 
>>    does not return the number of rows affected by each statement in the 
>> batch, 
>>    only the total number of rows affected by the batch.
>>    
>> Browsing the source files, found that in AbstractEntityPersister define 
>> the IsBatchable property looking at the optimistic lock strategy.
>>
>> Since all the updates done in batching, have the ID (and the VERSION), is 
>> not enough to have the total number of rows affected by the batch in order 
>> to know if all the updates were sucessfully done? Is it really necessary to 
>> have the rows affected by each statement of the batch?
>>
>> Any improvement to make possible to do the updates in batchs will really 
>> boost performance.
>>
>> Thank you very much !
>>
>> Tikuna
>>
>>
>>

-- 
You received this message because you are subscribed to the Google Groups 
"nhusers" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/nhusers.
For more options, visit https://groups.google.com/d/optout.

Reply via email to