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

freeze lock for update #4206

Closed
martingg88 opened this issue May 22, 2015 · 55 comments
Closed

freeze lock for update #4206

martingg88 opened this issue May 22, 2015 · 55 comments
Assignees
Labels
Milestone

Comments

@martingg88
Copy link

@Laa seem like it still lock when doing update/insert. here the thread dump for your debugging. In between we have some discussion at #4090

you can get the thread dump for your debugging.

https://drive.google.com/file/d/0B4HzIyaEAQ5WUDhoM2h4Sm56SDQ/view?usp=sharing

@andrii0lomakin andrii0lomakin added this to the 2.1 GA milestone May 22, 2015
@andrii0lomakin andrii0lomakin self-assigned this May 22, 2015
@lvca lvca modified the milestones: 2.1-rc3, 2.1 GA May 22, 2015
@andrii0lomakin
Copy link
Member

Do you have Skype so we may chat and debug or merey code to reproduce this issue ?

@martingg88
Copy link
Author

here is my skype ckgan2004@hotmail.com. what time you about want to do dis discussion?

@andrii0lomakin
Copy link
Member

what about now ))

@martingg88
Copy link
Author

sorry.. i can't make it now...how about 2 hour later at 21:00+0800 malaysia time?

@andrii0lomakin
Copy link
Member

2 hours later is ok for me

@martingg88
Copy link
Author

ok...see u later..

@andrii0lomakin
Copy link
Member

I can not find your contact in Skype, could you connect to me lomakin_andrey .

@martingg88
Copy link
Author

sure..i will do it later...

@lvca lvca modified the milestones: 2.1-rc3, 2.1 GA May 22, 2015
@martingg88
Copy link
Author

@Laa... as for last conversation over skype. are you able to reproduce this issue?

@andrii0lomakin
Copy link
Member

Hi, Sorry I am super busy by issue #4144 . Once I finish it I will be back
to this one .
On May 28, 2015 6:32 PM, "martingg88" notifications@github.com wrote:

@Laa https://github.com/laa... as for last conversation over skype. are
you able to reproduce this issue?


Reply to this email directly or view it on GitHub
#4206 (comment)
.

@martingg88
Copy link
Author

sure. thanks

@andrii0lomakin
Copy link
Member

Hi, coudl you try this distribution https://www.dropbox.com/s/xs2w59sncv9ln4m/orientdb-community-2.1-SNAPSHOT-distribution.zip?dl=0 . Does deadlock still persist ?

@martingg88
Copy link
Author

i will try it today or tomorrow and let you know the result. thanks.

@andrii0lomakin
Copy link
Member

Thank you for your help !

@martingg88
Copy link
Author

I still facing the issue as follows after tested.
this should be the concurrent issue for update.
OrientDB.RequestError: Error on saving record #59:11062

On Fri, Jun 5, 2015 at 9:55 PM, Andrey Lomakin notifications@github.com
wrote:

Thank you for your help !


Reply to this email directly or view it on GitHub
#4206 (comment)
.

@martingg88
Copy link
Author

I have the issue as follows after tested.
this happen for both execute update one after another and in parallel/concurrent.
OrientDB.RequestError: Error on saving record #59:11062

@andrii0lomakin
Copy link
Member

Could you send me stack trace from server log for this error ?

@andrii0lomakin
Copy link
Member

Is it correct that there is no freeze now, but there is issue which you reported ?

@martingg88
Copy link
Author

  1. this seems like another issue which relate to the following exception.
    com.orientechnologies.orient.core.exception.ODatabaseException: Error on
    saving record in cluster Define a method to call against the POJO after unmarshalling [moved] #56 [ONetworkProtocolBinary]
  2. I can't see any freeze there since i had have this issue ever.

On Mon, Jun 8, 2015 at 6:13 PM, Andrey Lomakin notifications@github.com
wrote:

Is it correct that there is no freeze now, but there is issue which you
reported ?


Reply to this email directly or view it on GitHub
#4206 (comment)
.

@lvca
Copy link
Member

lvca commented Jul 15, 2015

It's already released in 2.0.12 and 2.1-rc5

@martingg88
Copy link
Author

ok..will try it as soon as possible

@martingg88
Copy link
Author

In between, if i can't reply you in timely manner. You can ask @Laa to test it as he have my test script for two cases (etc: concurrent update and transaction roll back issue). Thanks.

@martingg88
Copy link
Author

test script working fine for orientjs- 2.0.0 and orientdb-2.1-rc5 enterprise version.

  1. test run update 1000 records with lock mechanism in concurrent/parallel.
    • passed without error.
    • it takes 2919ms to complete. 2919ms is acceptable performance to handle 1000 records?

@lvca
Copy link
Member

lvca commented Jul 31, 2015

How you did insert those records?

@martingg88
Copy link
Author

can you get the test script from @Laa. he have the test script for both update and commit in concurrent.
from the test scripts, you should be able to see how i update the records. thanks.

@lvca lvca modified the milestones: 2.1 GA, 2.1.1 Aug 5, 2015
@lvca lvca modified the milestones: 2.1.1, 2.1.x (next hotfix) Aug 31, 2015
@martingg88
Copy link
Author

any update?

@wolf4ood
Copy link
Member

wolf4ood commented Feb 2, 2016

@martingg88

have you solved this?

@martingg88
Copy link
Author

You should check with your team members please.

@lvca lvca assigned robfrank and unassigned wolf4ood May 13, 2016
@robfrank robfrank assigned wolf4ood and unassigned robfrank May 16, 2016
@wolf4ood
Copy link
Member

@martingg88

this should been fixed
can you confirm that?

@wolf4ood
Copy link
Member

@martingg88

i'm closing this. If you still have problem with this issue, you can reopen it

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

No branches or pull requests

6 participants