search

Syncleus / Ferma

An ORM / OGM for the TinkerPop graph stack.
http://syncleus.com/Ferma
Apache License 2.0
137 stars 26 forks source link

How to handle detached elements with Ferma? #49

Open malagus opened 6 years ago

malagus commented 6 years ago

TinkerPop supports re-attaching DetachedElement back to graph. It's possible to achieve something similar using ferma ? We using ferma in rest api so user can get some graph elements, modify them and send back so we need to update our graph element. Currently do doing it manually but possibility to re-attaching elements to graph can be very useful.

freemo commented 6 years ago

When you start manually do you mean your detaching and reattaching using the raw tinkerpop calls and then just wrapping/rewrapping with framed objects?

On Wed, May 16, 2018, 7:17 AM malagus notifications@github.com wrote:

TinkerPop supports re-attaching DetachedElement http://tinkerpop.apache.org/javadocs/3.2.2/full/org/apache/tinkerpop/gremlin/structure/util/detached/DetachedElement.html back to graph. It's possible to achieve something similar using ferma ? We using ferma in rest api so user can get some graph elements, modify them and send back so we need to update our graph element. Currently do doing it manually but possibility to re-attaching elements to graph can be very useful.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Syncleus/Ferma/issues/49, or mute the thread https://github.com/notifications/unsubscribe-auth/AC5JAp2XVPim-W3UqUIDxyQbLUwzp9X1ks5tzAqvgaJpZM4UBHbG .

malagus commented 6 years ago

something like that because we can't create instance of framed object without real graph

freemo commented 6 years ago

We havent addressed this explicitly. But I need this feature for my own software that uses Ferma as well. So I completely support extending the system to support this. Would you like to help contribute a PR for this or have any proposals for changes to the interface, or shall I spearhead that?

On Wed, May 16, 2018 at 7:56 AM, malagus notifications@github.com wrote:

something like that because we can't create instance of framed object without real graph

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Syncleus/Ferma/issues/49#issuecomment-389492672, or mute the thread https://github.com/notifications/unsubscribe-auth/AC5JAgRPMb661UX2HRaLctvZwB5Clk5mks5tzBP3gaJpZM4UBHbG .

malagus commented 6 years ago

I can contribute. We can discuss the requirements.

freemo commented 6 years ago

Wonderful, We can discuss here or in Gitter, which do you prefer?

On Fri, May 18, 2018 at 1:30 AM, malagus notifications@github.com wrote:

I can contribute. We can discuss the requirements.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Syncleus/Ferma/issues/49#issuecomment-390098889, or mute the thread https://github.com/notifications/unsubscribe-auth/AC5JAv1Tmg2aP5J1SBjqyzMs4RHSc1ncks5tzlyPgaJpZM4UBHbG .

malagus commented 6 years ago

We can discuss on Gitter

syncleus-bot commented 6 years ago

See you there

On Mon, May 21, 2018 at 4:42 AM, malagus notifications@github.com wrote:

We can discuss on Gitter

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Syncleus/Ferma/issues/49#issuecomment-390591515, or mute the thread https://github.com/notifications/unsubscribe-auth/AWNYbwvrbQaf1qZzWHosN2M2ggAXlCuaks5t0n3egaJpZM4UBHbG .