cdo remove delta [message #687146] |
Tue, 07 June 2011 04:40  |
Eclipse User |
|
|
|
Hi All,
We are using a version of CDO from a few months ago. We now run into the
following problem. The CDORemoveFeatureDeltaImpl is index based. How is
it possible to always remove the correct element when an update is
received from the server. At the moment I think sometimes the wrong
element is changed because the updates from the server has changed the
underlying list and applying the CDORemoveDelta removes the wrong element.
Is this code still used in the current version of CDO?
Thanks,
Ronald
|
|
|
|
Re: cdo remove delta [message #687148 is a reply to message #687147] |
Tue, 07 June 2011 05:06  |
Eclipse User |
|
|
|
Maybe I was not quite clear :)
The merge is using the CDORevisionMerger. The merge is at some point
executing
@Override
public void visit(CDORemoveFeatureDelta delta)
{
revision.remove(delta.getFeature(), delta.getIndex());
}
this code is executed after the updates received from the server are
processed. The index in the delta is not adjusted.
I did some tests on and this will definitely delete the wrong element.
However this code is deprecated(I think). So if that's the case we need
have to decide what to about do. Keep current CDO and try to fix or upgrade.
Thanks,
Ronald
On 7-6-2011 10:55, Eike Stepper wrote:
> Am 07.06.2011 10:40, schrieb Ronald Krijgsheld:
>>
>> Hi All,
>>
>> We are using a version of CDO from a few months ago. We now run into
>> the following problem. The CDORemoveFeatureDeltaImpl is index based.
> What exactly is that problem?
>
>> How is it possible to always remove the correct element when an update
>> is received from the server.
> Is that a question? I don't understand it ;-(
>
>> At the moment I think sometimes the wrong element is changed because
>> the updates from the server has changed the underlying list and
>> applying the CDORemoveDelta removes the wrong element.
> What makes you think that?
>
>>
>> Is this code still used in the current version of CDO?
> You neither specified what code nor what version you're using exactly.
> It's best that you use Eclipse's team functions to compare the piece of
> code you're looking at.
>
> Cheers
> /Eike
>
> ----
> http://www.esc-net.de
> http://thegordian.blogspot.com
> http://twitter.com/eikestepper
>
>
|
|
|
Re: cdo remove delta [message #687393 is a reply to message #687146] |
Tue, 07 June 2011 04:55  |
Eclipse User |
|
|
|
Am 07.06.2011 10:40, schrieb Ronald Krijgsheld:
>
> Hi All,
>
> We are using a version of CDO from a few months ago. We now run into the following problem. The CDORemoveFeatureDeltaImpl is index based.
What exactly is that problem?
> How is it possible to always remove the correct element when an update is received from the server.
Is that a question? I don't understand it ;-(
> At the moment I think sometimes the wrong element is changed because the updates from the server has changed the underlying list and applying the CDORemoveDelta removes the wrong element.
What makes you think that?
>
> Is this code still used in the current version of CDO?
You neither specified what code nor what version you're using exactly. It's best that you use Eclipse's team functions to compare the piece of code you're looking at.
Cheers
/Eike
----
http://www.esc-net.de
http://thegordian.blogspot.com
http://twitter.com/eikestepper
|
|
|
Re: cdo remove delta [message #687394 is a reply to message #687147] |
Tue, 07 June 2011 05:06  |
Eclipse User |
|
|
|
Maybe I was not quite clear :)
The merge is using the CDORevisionMerger. The merge is at some point
executing
@Override
public void visit(CDORemoveFeatureDelta delta)
{
revision.remove(delta.getFeature(), delta.getIndex());
}
this code is executed after the updates received from the server are
processed. The index in the delta is not adjusted.
I did some tests on and this will definitely delete the wrong element.
However this code is deprecated(I think). So if that's the case we need
have to decide what to about do. Keep current CDO and try to fix or upgrade.
Thanks,
Ronald
On 7-6-2011 10:55, Eike Stepper wrote:
> Am 07.06.2011 10:40, schrieb Ronald Krijgsheld:
>>
>> Hi All,
>>
>> We are using a version of CDO from a few months ago. We now run into
>> the following problem. The CDORemoveFeatureDeltaImpl is index based.
> What exactly is that problem?
>
>> How is it possible to always remove the correct element when an update
>> is received from the server.
> Is that a question? I don't understand it ;-(
>
>> At the moment I think sometimes the wrong element is changed because
>> the updates from the server has changed the underlying list and
>> applying the CDORemoveDelta removes the wrong element.
> What makes you think that?
>
>>
>> Is this code still used in the current version of CDO?
> You neither specified what code nor what version you're using exactly.
> It's best that you use Eclipse's team functions to compare the piece of
> code you're looking at.
>
> Cheers
> /Eike
>
> ----
> http://www.esc-net.de
> http://thegordian.blogspot.com
> http://twitter.com/eikestepper
>
>
|
|
|
Powered by
FUDForum. Page generated in 0.14903 seconds