-
Notifications
You must be signed in to change notification settings - Fork 54
Session + Spark example #1022
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
Session + Spark example #1022
Conversation
✅ Deploy Preview for neo4j-graph-data-science-client canceled.
|
e511a66 to
95a787a
Compare
|
Closes #991 |
| "# Create a GDS session!\n", | ||
| "gds = sessions.get_or_create(\n", | ||
| " # we give it a representative name\n", | ||
| " session_name=\"bike_trips\",\n", |
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 read about bike trips just today in our logs. nice to learn its your workload 👀
Mats-SX
left a comment
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 have many textual changes on the notebook.
I think this is a great step forward! I think Nathan and Stu will be very happy to see it.
| "Flight returned internal error, with message: org.apache.arrow.flight.FlightRuntimeException: ", "" | ||
| ) | ||
| improved_message = improved_message.replace( | ||
| "Failed to invoke procedure `gds.arrow.project`: Caused by: org.apache.arrow.flight.FlightRuntimeException: ", |
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.
this procedure doesn't really exist anymore -- only in very old Neo4j versions. these days it's called gds.arrow.project.v2 or, more commonly v3
you just moved it, I realise, but I want to note the ineffectiveness here.
src/graphdatascience/procedure_surface/arrow/catalog/catalog_arrow_endpoints.py
Show resolved
Hide resolved
Co-authored-by: Florentin Dörre <florentin.dorre@neo4j.com>
Co-authored-by: Mats Rydberg <mats@neo4j.org>
9b0bfe8 to
9569339
Compare
ref GDSA-373
ref GDSA-469