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

Support cluster selection for DELETE commands #6590

Closed
5 tasks done
StarpTech opened this issue Aug 19, 2016 · 8 comments
Closed
5 tasks done

Support cluster selection for DELETE commands #6590

StarpTech opened this issue Aug 19, 2016 · 8 comments
Assignees
Milestone

Comments

@StarpTech
Copy link

StarpTech commented Aug 19, 2016

OrientDB Version, operating system, or hardware.

  • v2.0 SNAPSHOT[ ] - .18[ X] .17[ X] .16[ ] .15[ ] .14[ ] .13[ ] .12[ ] .11[ ] .10[ ] .9[ ] .8[ ] .7[ ] .6[ ] .5[ ] .4[ ] .3[ ] .2[ ] .1[ ] .0[ ]
  • v2.1 SNAPSHOT[ ] - .16[X ] .15[ X] .14[ X] .13[ ] .12[ ] .11[ ] .10[ ] .9[ ] .8[ ] .7[ ] .6[ ] .5[ ] .4[ ] .3[ ] .2[ ] .1[ ] .0[ ]
  • v2.2 SNAPSHOT[ ] - .rc1[X ] .beta2[ X] .beta1[X ]

All Versions!

Operating System

  • Linux
  • MacOSX
  • Windows
  • Other Unix
  • Other, name?

Expected behavior and actual behavior

With regard to the Distribution strategy of orientdb it must be possible to select a cluster to avoid a scan through all irrelevant clusters.

Expected sql support:

DELETE EDGE <class> CLUSTER <cluster>|[<cluster>,...] WHERE <where>

Steps to reproduce the problem

throws an exception:

DELETE EDGE <class> CLUSTER <cluster>|[<cluster>,...] WHERE <where>
@smolinari
Copy link
Contributor

Hey Dustin,

Doesn't the simpler command DROP CLUSTER <cluster-name>|<cluster-id> work?

Scott

@StarpTech
Copy link
Author

Hi @smolinari I think yes but it has nothing to do with it. I want to remove a record from a specific cluster.

@smolinari
Copy link
Contributor

Oh boy. Sorry about that. Don't know what I was thinking......

Scott

@lvca
Copy link
Member

lvca commented Aug 22, 2016

@StarpTech you can always delete edges by using rids. This is fast and is executed only by the servers that have the record:

DELETE EDGE #13:4343

or

DELETE EDGE [#13:4343, #134:434, #32:965976]

@lvca
Copy link
Member

lvca commented Aug 22, 2016

I've just checked and DELETE EDGE command is not optimized for this, but DELETE VERTEX contains it. I'm converting this in an enhancement.

@lvca lvca self-assigned this Aug 22, 2016
lvca added a commit that referenced this issue Aug 22, 2016
lvca added a commit that referenced this issue Aug 22, 2016
@lvca
Copy link
Member

lvca commented Aug 22, 2016

Implemented in 2.2.x and develop branches.

@StarpTech
Copy link
Author

StarpTech commented Aug 23, 2016

Hi @lvca thank you but can you port it to the 2.1.x series? You know my issues with switching to the 2.2.x series. thank you.

@VillHub
Copy link

VillHub commented Aug 30, 2016

DELETE EDGE edgename
not working when E is orestricted subclass
issue #6425

@lvca lvca added this to the 3.0 milestone Aug 5, 2017
@lvca lvca assigned luigidellaquila and unassigned lvca Aug 5, 2017
@lvca lvca modified the milestones: 3.0.0, 3.0.x Apr 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

7 participants