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

Allow update edge in/out properties with select query #5384

Merged

Conversation

matanshukry
Copy link
Contributor

Related to #1114 / #4436

p.s.
Currently there seems to be a memory leak in the branch, regardless of my commit.
I'll merge, fix if needed, commit and push when possible.

@luigidellaquila
Copy link
Member

Hi @matanshukry

thank you very much, the PR is absolutely correct. There is just a problem with the build, we are fixing it right now.
I'll re-run the build and will merge the PR as soon as the problem is fixed.

Thanks

Luigi

@lvca lvca modified the milestones: 2.2.0-beta, 2.2.0-rc1 Dec 13, 2015
@matanshukry
Copy link
Contributor Author

Merged with develop again, all checks seems to pass now

@lvca
Copy link
Member

lvca commented Jan 1, 2016

Thank you!

lvca added a commit that referenced this pull request Jan 1, 2016
Allow update edge in/out properties with select query
@lvca lvca merged commit ba39419 into orientechnologies:develop Jan 1, 2016
@matanshukry matanshukry deleted the issueUpdateEdgeInnerSelect branch January 1, 2016 23:25
@lvca lvca modified the milestones: 2.2.0-rc1, 2.2.0-beta Jan 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants