Skip to content
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

Azure table : concurrent update strange behavior #259

Open
pabeucher opened this issue Sep 30, 2020 · 3 comments · May be fixed by #392
Open

Azure table : concurrent update strange behavior #259

pabeucher opened this issue Sep 30, 2020 · 3 comments · May be fixed by #392
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed

Comments

@pabeucher
Copy link

I'm submitting a...


[ ] Regression 
[X ] Bug report
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request => Please do not submit support request here, instead post your question on Stack Overflow.

Current behavior

We have one table with several entities
Some entities have the same partitionKey.
Each entities have a unique RowKey

When we process two updates at the same time with same PartitionKey but different RowKey we have a strange behavior => the two entities are mixed with same content

I noticed the lib use azure replaceEntity to do an update. When testing same case with Microsoft lib and mergeEntity function we don t have the issue, but I would prefer to use nest lib as it’s integration is more interesting with Nest

Expected behavior

Each updated entities are correctly and independently updated

Minimal reproduction of the problem with instructions

Create to update at the same time

What is the motivation / use case for changing the behavior?

Environment


Nest version: 6.9 to 7.0

 
For Tooling issues:
- Node version: 10.14 and + 
- Platform:  Mac. Linux and Windows

Others:
Package manager : npm
@manekinekko manekinekko added bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed labels Nov 9, 2020
@ayshiff
Copy link

ayshiff commented Nov 9, 2020

@manekinekko Can I work on this ? 👍

@ayshiff
Copy link

ayshiff commented Nov 12, 2020

@manekinekko What are your thoughts on this ?

I did find some ressources on concurrency but nothing really helpful for this case.
Do you have any idea where it could come from ?

@manekinekko
Copy link
Collaborator

@ayshiff sorry about the delay. I haven't seen the notification for this issue.

If you are still investigating this, maybe you can try what @pabeucher said and replace replaceEntity with mergeEntity.

@ayshiff ayshiff linked a pull request Apr 2, 2021 that will close this issue
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants