java-rest-binding icon indicating copy to clipboard operation
java-rest-binding copied to clipboard

Published 20 hours ago • verified publisher icon neo4j-contrib

Added cypher support

Open nrkkalyan opened this issue 12 years ago • 7 comments

Hi,

I have added few method in java-rest-binding to make it work with neoclipse. Please find my changes and let me know if there is something wrong.

nrkkalyan avatar Apr 21 '12 14:04 nrkkalyan

Did you look into rewriting this so that it creates a correct rest-traversal description and sends it over to the server?

jexp avatar May 04 '12 17:05 jexp

Hi,

Yes I did tried, I emailed to you on Apr 21, 2012 at 7:07 PM ,anyway. Here is my findings,

Can you please let me know if we should try to something similar to OldTraverserWrapper but using RestTraversal.description() or something different.

As I tried something but finally stuck between _org.neo4j.graphdb.Traverser_and org.neo4j.graphdb.traversal.Traverser Since node.traverse() returns org.neo4j.graphdb.Traverser, *I am not getting what to modify.

Kalyan *

On Fri, May 4, 2012 at 7:32 PM, Michael Hunger < [email protected]

wrote:

Did you look into rewriting this so that it creates a correct rest-traversal description and sends it over to the server?

Reply to this email directly or view it on GitHub: https://github.com/neo4j/java-rest-binding/pull/3#issuecomment-5515749

Thanks and Regards N Radhakrishna Kalyan

nrkkalyan avatar May 05 '12 07:05 nrkkalyan

Have you looked into upgrading neoclipse to the new traversal API ? (Or even better use cypher in these places)

That would help a lot.

Michael

jexp avatar May 05 '12 07:05 jexp

No, But that is in my road map. I will start on this soon.

Kalyan

On Sat, May 5, 2012 at 9:36 AM, Michael Hunger < [email protected]

wrote:

Have you looked into upgrading neoclipse to the new traversal API ? (Or even better use cypher in these places)

That would help a lot.

Michael

Reply to this email directly or view it on GitHub: https://github.com/neo4j/java-rest-binding/pull/3#issuecomment-5525172

Thanks and Regards N Radhakrishna Kalyan

nrkkalyan avatar May 05 '12 08:05 nrkkalyan

this may be interesting: https://github.com/FaKod/neo4j-scala/blob/AddNodeTraverse/src/main/java/org/neo4j/rest/graphdb/traversal/RestOldTraverserWrapper.java

FaKod avatar May 10 '12 15:05 FaKod

Nice Impl, but it still does the filtering on the client side, right?

jexp avatar May 10 '12 16:05 jexp

Yes, partly.

I tweaked it a little bit to support neo4j-scala and its typed traverser.

See the tests here: https://github.com/FaKod/neo4j-scala/blob/AddNodeTraverse/src/test/scala/org/neo4j/scala/unittest/TypedTraverserRESTSpec.scala Am 10.05.2012 18:38 schrieb "Michael Hunger" < [email protected]

:

Nice Impl, but it still does the filtering on the client side, right?

Reply to this email directly or view it on GitHub: https://github.com/neo4j/java-rest-binding/pull/3#issuecomment-5630033

FaKod avatar May 10 '12 16:05 FaKod