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.

Sign up for GitHub

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

Jump to bottom

Make tinkerpop Gaffer operation logging configurable #2994

Closed
t92549 opened this issue Jun 28, 2023 · 0 comments · Fixed by #3180
Closed

Make tinkerpop Gaffer operation logging configurable #2994

t92549 opened this issue Jun 28, 2023 · 0 comments · Fixed by #3180
Assignees
@cn337131
Labels
enhancement Improvement to existing functionality/feature tinkerpop Specific to/touches the tinkerpop module
Milestone
v2.2.1

Comments

@t92549
Copy link
Contributor

t92549 commented Jun 28, 2023

Describe the new feature you'd like
As discussed in #2984 (comment), we should make the logging of Gaffer Operations in tinkerpop configurable and not active by default.

Why do you want this feature?
It would clear up the logs in normal usage.
@t92549 t92549 added enhancement Improvement to existing functionality/feature tinkerpop Specific to/touches the tinkerpop module labels Jun 28, 2023
@GCHQDeveloper314 GCHQDeveloper314 added this to the Backlog milestone Jul 7, 2023
@cn337131 cn337131 self-assigned this Mar 26, 2024
@github-actions github-actions bot mentioned this issue Mar 26, 2024
Merged
@tb06904 tb06904 modified the milestones: Backlog, v2.2.1 Apr 5, 2024
GCHQDeveloper314 added a commit that referenced this issue Apr 8, 2024
@GCHQDeveloper314
Merge branch 'develop' into gh-2994-make-gremlin-logging-configurable
cfc3186
GCHQDeveloper314 pushed a commit that referenced this issue Apr 8, 2024
@cn337131
Gh-2994: Make gremlin logging configurable (#3180)
fe1d93a 
* only show info logs if they are enabled

* move check and fix formatting

* change info to debug
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cn337131 cn337131

Labels
enhancement Improvement to existing functionality/feature tinkerpop Specific to/touches the tinkerpop module
Projects
None yet
Milestone
v2.2.1
Development

Successfully merging a pull request may close this issue.

Gh-2994: Make gremlin logging configurable gchq/Gaffer
4 participants
@t92549 @GCHQDeveloper314 @tb06904 @cn337131