-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Make query result push rows to the server response #4345
Conversation
@@ -227,15 +227,17 @@ | |||
/** | |||
* This is the visitor interface you need to implement to use the {@link Result#accept(ResultVisitor)} method. | |||
*/ | |||
interface ResultVisitor | |||
interface ResultVisitor<VisitationException extends Exception> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we have something like
interface ResultsVisitor<E> extends org.neo4j.helpers.collection.Visitor<Result.ResultRow, E>
instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd prefer not. It would be nice if we could treat the org.neo4j.helpers
package as internal and not expose it in the API.
a7ee4f7
to
02effe1
Compare
02effe1
to
90e0a7a
Compare
@@ -22,35 +22,21 @@ package org.neo4j.cypher.internal.compiler.v2_3.birk.il | |||
import org.neo4j.graphdb.Direction | |||
|
|||
case class ExpandC(fromVar: String, relVar: String, dir: Direction, types: Map[String, String], toVar: String, inner: Instruction) extends LoopDataGenerator { | |||
private val visitMethod = | |||
private val theBody = |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I really the changes in this file, but I believe it might not really belong in this commit, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It was required to make the tests pass...
I thought of putting in a separate commit in the same PR, but decided against it since it would have made the first commit useless on its own.
Make query result push rows to the server response
No description provided.