Added cypher support

[nrkkalyan]

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.

[jexp]

Imho this old-traverser does not the correct thing as it would execution the traversal over the wire for each step, it should rather use RestTraversal.description() to build up the query (in a similar fashion), which would be executed on the server-side.

[jexp]

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

[nrkkalyan]

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 <
reply@reply.github.com

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:
#3 (comment)

Thanks and Regards
N Radhakrishna Kalyan

[jexp]

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

[nrkkalyan]

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 <
reply@reply.github.com

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:
#3 (comment)

Thanks and Regards
N Radhakrishna Kalyan

[FaKod]

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

[jexp]

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

[FaKod]

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" <
reply@reply.github.com

:

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

---

Reply to this email directly or view it on GitHub:
#3 (comment)