Skip to content

Added synchronized to createNode and other methods - #129

Closed
casalribeiro wants to merge 1 commit into
extent-framework:masterfrom
casalribeiro:bugfix/mcampos/createNode-synchronism
Closed

Added synchronized to createNode and other methods#129
casalribeiro wants to merge 1 commit into
extent-framework:masterfrom
casalribeiro:bugfix/mcampos/createNode-synchronism

Conversation

@casalribeiro

@casalribeiro casalribeiro commented Feb 17, 2020

Copy link
Copy Markdown

Fix issue with replaced test case (node) when running parallel tests

@casalribeiro casalribeiro mentioned this pull request Feb 17, 2020
Closed
@anshooarora

anshooarora commented Apr 9, 2020

Copy link
Copy Markdown
Member

@casalribeiro createNode does require to be synchronized, but why do the assignment of meta/attributes require this since these are added to self.

1 2 3 4

@anshooarora

anshooarora commented Apr 9, 2020

Copy link
Copy Markdown
Member

I have synchronized createNode here

@casalribeiro

casalribeiro commented Apr 24, 2020

Copy link
Copy Markdown
Author

Fixed in version 4.1.5

@casalribeiro
casalribeiro deleted the bugfix/mcampos/createNode-synchronism branch April 24, 2020 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@anshooarora anshooarora Awaiting requested review from anshooarora

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@casalribeiro @anshooarora