Graphql logging - Githubissues

Pometry / Raphtory

Scalable graph analytics database powered by a multithreaded, vectorized temporal engine, written in Rust

https://raphtory.com

GNU General Public License v3.0

336 stars 54 forks source link

Graphql logging #1746

Closed miratepuffin closed 2 weeks ago

miratepuffin commented 4 weeks ago

What changes were proposed in this pull request?

Fixed tracing within the graphql server
Pushed logging throughout raphtory - removing all printlns
Fixed a constant properties function in graphql which was pub instead of async

Why are the changes needed?

Logging and tracing make it a lot easier to see what is going on in the server

Does this PR introduce any user-facing change? If yes is this documented?

No

How was this patch tested?

Current test suite

Issues

Fixes #1697

Are there any further changes required?

fabianmurariu commented 2 weeks ago

This was a lot of work, but I can't help but thinking 90% of these println! should have been removed instead of replaced