gchq / Gaffer

A large-scale entity and relation database supporting aggregation of properties
Apache License 2.0
1.77k stars 350 forks source link

Gh-2994: Make gremlin logging configurable #3180

Closed cn337131 closed 6 months ago

cn337131 commented 6 months ago

Costly logging should only display now if the SLF4J log level is enabled.

Related issue

codecov[bot] commented 6 months ago

Codecov Report

Attention: Patch coverage is 14.28571% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 66.68%. Comparing base (7a34d83) to head (d62c374).

Files Patch % Lines
...a/uk/gov/gchq/gaffer/tinkerpop/GafferPopGraph.java 14.28% 5 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #3180 +/- ## ============================================= - Coverage 66.71% 66.68% -0.03% Complexity 2555 2555 ============================================= Files 907 907 Lines 29060 29061 +1 Branches 3243 3242 -1 ============================================= - Hits 19386 19380 -6 - Misses 8238 8244 +6 - Partials 1436 1437 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.