8000 Allow addEdge() to take vertex ids in the JS library by moonglum · Pull Request #528 · arangodb/arangodb · GitHub
[go: up one dir, main page]
Skip to content

Allow addEdge() to take vertex ids in the JS library #528

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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy 8000 statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 2,481 commits into from
Closed

Allow addEdge() to take vertex ids in the JS library #528

wants to merge 2,481 commits into from

Conversation

moonglum
Copy link

Currently, this works:

v1 = graph.addVertex();
v2 = graph.addVertex();

z = graph.addEdge(v1, v2, null, {"name":"john smith"});

However, sometimes, I have the ID of a vertex, but not a reference to it. I can of course, ask ArangoDB to retrieve the vertex and then use it with addEdge(), but that's not efficient.

I would love to be able to do this:

z = graph.addEdge('myvertexcollection/123', 'myvertexcollection/456', null, {"name":"john smith"});

@frankmayer
Copy link
Contributor

Oh yes, that would be great 👍

@ghost ghost assigned fceller May 16, 2013
@ghost ghost assigned moonglum Jun 4, 2013
hkernbach and others added 27 commits October 8, 2013 21:56
@hkernbach
more transparency, new api style
c3f6e44
@hkernbach
more transparency, new api style
8168520
@hkernbach
dashboard label fix for detailedgraph
797cbc4
@hkernbach
css fixes for apps-view and docs-view
4044863
@hkernbach
foxxcss
132032c
@jsteemann
keep deletions in compaction
2eb9ff1
@jsteemann
added assertion
4cac289
@jsteemann
added config file for arangodump
5d5624a
@jsteemann
fix filename
77bb297
@jsteemann
removed superfluous parameter
980ccb6
@jsteemann
added collection id translation for dumps
7cf6198
@jsteemann
fix temp file creation on Windows
fff528b
@jsteemann
fix some VS warnings
473bf9d
@jsteemann
fix Windows file handling
bd2d2c3
@jsteemann
refuse to start if no app path is set
b509e79
@jsteemann
Windows fixes
49b6f64
@jsteemann
updated config files
52f480f
@jsteemann
config file handling
f8cba10
@jsteemann
updated Windows batch files
937415f
@jsteemann
updated batch files
baef8f2
@jsteemann
notes on Windows and upgrading
d950d4f
@jsteemann
fixed typo
9e88528
@jsteemann
worked around arguments.callee
47808ee
@jsteemann
use batches in arangodump
1536caa
@jsteemann
better not use ofstream
55ab125
8000 @jsteemann
resolve collection names on reimport
1427daf
@jsteemann
updated arangorestore
9b3705a
jsteemann and others added 24 commits October 30, 2013 01:33
@jsteemann
removed --log.format command line option
976ee76
@jsteemann
issue #424: Documentation about IDs needs to be upgraded
1f3f4ea
@jsteemann
updated docs
2842eb9
@jsteemann
removed eol option
226679f
@jsteemann
added pretty print option
f86f837
@jsteemann
added help text
8cb3434
@jsteemann
print error details
e7cdb16
@jsteemann
show error details in arangoimp
c95da02
@jsteemann
issue #642: renamed macro from TRI_WARN_UNUSED to TRI_WARN_UNUSED_RESULT
80f9437
@jsteemann
improve error message
ee11fde
@jsteemann
made AQL DOCUMENT function polymorphic
28d27a4
@jsteemann
issue #597: added more documentation
63a2be9
@jsteemann
allow dynamic parameters in DOCUMENT()
3c1fdbb
@jsteemann
fixed --javascript.check option for arangosh
f1f2070
@jsteemann
more help
2dc61d8
@luebbert42
Paragraph on Foxx added
267ba59
@jsteemann
Merge pull request #649 from luebbert42/devel
a1ca973 
Paragraph on Foxx added
@jsteemann
issue #650: Randonmess of any() should be improved
6d050b6
@jsteemann
windows fixes
715f795
@jsteemann
updated Windows msi
80d193e
Whitespace.
49350d9
Graph Module / Client: It has to send the ID, not the key
e7e47ce
Code Style
d9a14a8
Graph Refactoring: Determination of ID pushed to Common
< 8000 a href="/arangodb/arangodb/pull/528/commits/1c2b379b816c8e720d805848523ec23e3168bee6" class="Link--secondary">1c2b379
@moonglum moonglum closed this Oct 31, 2013
@moonglum
Copy link

Sorry, accidentally chose the wrong target branch 😦

@moonglum
Copy link

See #651 – I'm really sorry.

@moonglum moonglum mentioned this pull request Oct 31, 2013
Merged
@moonglum
Copy link

Done 😉

@KVS85 KVS85 mentioned this pull request Mar 22, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@moonglum moonglum

Labels
1 Feature 3 API HTTP / Driver API related 3 Graph Graph query engine
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@moonglum @frankmayer @fceller @hkernbach @jsteemann @mchacki @luebbert42
0