Skip to content
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

Add ability to load graphs server-side using custom functions #8

Merged

Conversation

rlratzel
Copy link
Contributor

@rlratzel rlratzel commented May 16, 2022

Code and tests for allowing clients to load graphs on the GaaS server using custom functions. This allows a complex graph creation process to be used in order to create the graph a server would manage.

closes #4

@rlratzel rlratzel added improvement Improves an existing functionality non-breaking Introduces a non-breaking change python labels May 16, 2022
@rlratzel rlratzel added this to the 22.06 milestone May 16, 2022
@rlratzel rlratzel self-assigned this May 16, 2022
rlratzel added 5 commits May 16, 2022 23:48
…d tests to use that to determine if the server was ready, updated tests to cover preloaded extensions from the server CLI.
… end-to-end test and not just a unit test for the client API, made changes to allow tests to run without setting PYTHONPATH.
@rlratzel rlratzel marked this pull request as ready for review May 18, 2022 16:57
rlratzel added 3 commits May 18, 2022 12:15
…owing thriftpy2.transport.TTransportError to be raised so users do not need to import thriftpy2 to catch exceptions from GaaS.
Copy link
Member

@alexbarghi-nv alexbarghi-nv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rlratzel rlratzel merged commit 1ecf61a into rapidsai:branch-22.06 May 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improves an existing functionality non-breaking Introduces a non-breaking change python
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add ability to execute arbitrary data loading and ETL operations on graph input data
2 participants