Skip to content

Conversation

@DarthMax
Copy link
Contributor

@DarthMax DarthMax commented Nov 28, 2025

ref GDSA-373
ref GDSA-469

@netlify
Copy link

netlify bot commented Nov 28, 2025

Deploy Preview for neo4j-graph-data-science-client canceled.

Name Link
🔨 Latest commit 8a3a49b
🔍 Latest deploy log https://app.netlify.com/projects/neo4j-graph-data-science-client/deploys/69395a3c5f4d0500080d557b

@DarthMax DarthMax force-pushed the sessions_expose_arrow_client branch from e511a66 to 95a787a Compare November 28, 2025 15:28
@Mats-SX
Copy link
Contributor

Mats-SX commented Dec 1, 2025

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",
Copy link
Contributor

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 Mats-SX self-assigned this Dec 8, 2025
Copy link
Contributor

@Mats-SX Mats-SX left a 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: ",
Copy link
Contributor

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.

@DarthMax DarthMax force-pushed the sessions_expose_arrow_client branch from 9b0bfe8 to 9569339 Compare December 10, 2025 10:41
@DarthMax DarthMax merged commit 3dfae11 into neo4j:main Dec 10, 2025
7 checks passed
@DarthMax DarthMax mentioned this pull request Dec 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants