> No Jiri, i mean, when we have ContinueOnError = true , and if we throw any
> exception or not, doesn't matter, we should call AcceptChanges (of course if
> AcceptChangesDuringUpdate = true) only for those rows, that have no error.
> As i said, imagine you have ContinueOnError=true and 2 rows violate
> concurrency condition, after calling Update you will know, that 2 rows have
> errors (they will have HasError=true) but you won't be able to let the user
> correct situation, because you don't know anymore, where those 2 rows
> supposed to be updated or deleted when they caused concurrency violation -
> cause they have RowVersion = Unchanged...Actually when i think about it...it
> is not big problem either, because concurrency exception is important only
> for updates (where user can insist on own changes or refresh own data with
> changes committed to database before him). If you get concurrency exception
> on delete statement you cannot do anything anymore (but may be you can :) )
> . Anyway, i think it might be useful to have the rowversion preserved for
> erroneous rows, what the point of calling acceptchanges for rows that have
> errors and won't be committed in present state to database? What i'm talkin
> about actually this line of code
>
> if (this.AcceptChangesDuringUpdate)
> {
>      row.AcceptChanges();
> }
>
> i personally would prefer to have
>
> if (!row.HasErrors && this.AcceptChangesDuringUpdate)
> {
>      row.AcceptChanges();
> }

Oh, now I understand. I was almost sleeping, so I was thinking wrong.
So the main idea behind this is, "when I get error, let the row
untouched (without AcceptChanges) so I'll be able to repeat the call
to DB", right?

> But i'm not so skilled and experienced as you are, that's why i ask what do
> you think about this damn concurrency :)

No problem. Discussing problems brings us better provider. :)

>
> P.S. Jiri, can you tell me, why do i get
> Error: PROPFIND request failed on '/viewvc/firebird/NETProvider/trunk'
> Error: PROPFIND of '/viewvc/firebird/NETProvider/trunk': 301 Moved
> Permanently (http://firebird.svn.sourceforge.net)
> every time i try to check out from
> http://firebird.svn.sourceforge.net/viewvc/firebird/NETProvider/trunk/. i
> tried it from several places (with or without proxy) - hopeless :(

The http://firebird.svn.sourceforge.net/viewvc/firebird/NETProvider/
is only proxy for web-viewing.

-- 
Jiri {x2} Cincura (Microsoft Student Partner)
http://blog.vyvojar.cz/jirka/ | http://www.ID3renamer.com

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
Firebird-net-provider mailing list
Firebird-net-provider@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/firebird-net-provider

Reply via email to