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

Research Rust Driver #262

Open
JoshInnis opened this issue Sep 6, 2022 · 6 comments
Open

Research Rust Driver #262

JoshInnis opened this issue Sep 6, 2022 · 6 comments
Assignees
Labels
documentation Improvements or additions to documentation enhancement New request question Further information is requested

Comments

@JoshInnis
Copy link
Contributor

No description provided.

@JoshInnis JoshInnis moved this to Todo in Apache AGE Drivers Sep 6, 2022
@JoshInnis JoshInnis added documentation Improvements or additions to documentation enhancement New request question Further information is requested labels Sep 7, 2022
@Dzordzu
Copy link
Contributor

Dzordzu commented Sep 16, 2022

What do you need to know?

@Dzordzu
Copy link
Contributor

Dzordzu commented Sep 16, 2022

Maintainer information

Merging

I will merge rust driver as soon as I'm sure the code quality is acceptable. Furthermore, the driver needs to cover following functionalities

Driver will be merged as soon as it reaches the 1.0.0 version

Functionalities

  • Cypher query building aka Serializers v0.4.0
  • Enhaced support for prepared statements: prepare_cypher(q: cypher_query) method v0.5.0
  • Support for default graph in execute/query. Depends on Using postgres variables within cypher function #302
  • Tests. More tests
  • Security verification and documentation
  • Code cleanup (async for example)
  • Stabilized and documented API

Covered functionalities

  1. Connection to the age graph
  2. Pure cypher queires
  3. Unique indexes creation
  4. Constraints creation
  5. Asynchronous client (tokio)
  6. Implemented AgType structure

Usage

Covered in docs, and readme

Known issues

Driver allows to return only a single record (as the AgType)

As long as you can use a standard query() method this is not a high priority issue

Driver Versioning

  • until 1.0.0 versioning is very loose
  • after 1.0.0 versioning has to follow SemVer

@JoshInnis JoshInnis moved this from Todo to In Progress in Apache AGE Drivers Sep 16, 2022
@Dzordzu
Copy link
Contributor

Dzordzu commented Sep 16, 2022

Also, can anyone close this dangling issue?

@JoshInnis
Copy link
Contributor Author

The PMC members of the project have the ability

@Dzordzu
Copy link
Contributor

Dzordzu commented Sep 16, 2022

If you want, we can always contact via communicator of your choice and talk about details and your vision over driver

@lukehinds
Copy link
Contributor

@JoshInnis I can see unit tests are in now: https://github.com/Dzordzu/rust-apache-age/tree/master/tests

What is required for:

  • Security verification and documentation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New request question Further information is requested
Projects
No open projects
Status: In Progress
Development

No branches or pull requests

3 participants