New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Verify and Remove Edm.ConcurrencyMode #564

Closed
xuzhg opened this Issue May 17, 2016 · 1 comment

Comments

Projects
None yet
2 participants
@xuzhg
Member

xuzhg commented May 17, 2016

In What's new in OData Version 4

It says:

4.3.4 Pruned: ConcurrencyMode

This was a nice idea, but somewhat inflexible, so it has been replaced with an annotation using the term Core.OptimisticConcurrencyControl.

However, in ODL for v4, we still have EdmCurrencyMode definition, see

public EdmConcurrencyMode ConcurrencyMode
{
get { return this.concurrencyMode; }
}

So, please verify and remove it.

Assemblies affected

*OData .Net lib 7.x

Reproduce steps

None

Expected result

Removed

Actual result

No removed

Additional details

@xuzhg xuzhg added this to the 7.0 milestone May 17, 2016

@xuzhg xuzhg added ready labels May 17, 2016

@TomDu TomDu self-assigned this Jun 6, 2016

@TomDu TomDu added in-progress and removed ready labels Jun 8, 2016

@markdstafford markdstafford changed the title from Verify and Remove Edm.CurrencyMode to Verify and Remove Edm.ConcurrencyMode Jun 20, 2016

TomDu added a commit to TomDu/odata.net that referenced this issue Jun 21, 2016

TomDu added a commit to TomDu/odata.net that referenced this issue Jun 21, 2016

@TomDu

This comment has been minimized.

Contributor

TomDu commented Jun 21, 2016

There remains some code logic in test framework that handles CurrencyMode, will remove them later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment