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

[Improvement]: Application Logger log level setting #16940

Open
IronSean opened this issue Apr 16, 2024 · 1 comment
Open

[Improvement]: Application Logger log level setting #16940

IronSean opened this issue Apr 16, 2024 · 1 comment

Comments

@IronSean
Copy link
Contributor

Improvement description

After looking through the source code, and trying to use bin/console debug:config PimcoreApplicationLoggerBundle (only to see that "The extension with alias "pimcore_application_logger" does not have its getConfiguration() method setup. "), I can't see any easy way to configure a logging level for the application logger.

It would be useful to be able to set the minimum logging level so that anything calling the Application Logger will only log for Warning level or above. Especially for large datahub jobs this would allow to only log problematic rows instead of all rows. Dealing with multiple millions of rows of imports in datahub I've both created 300GB of application log tables and used up all available inodes on 500 GB data disk with the datahub file object files from the logging.

I've taken to manually truncating the log tables and purging the fileObjects because the lack of index on the level field in the application_logger table makes trying to preserve warning and error level logs too time consuming. Being able to filter so info level logs aren't made in the first place, but then be able to turn them on for additional logging when needed, would be very useful.

Copy link

github-actions bot commented May 7, 2024

Thanks a lot for reporting the issue. We did not consider the issue as "Pimcore:Priority", "Pimcore:ToDo" or "Pimcore:Backlog", so we're not going to work on that anytime soon. Please create a pull request to fix the issue if this is a bug report. We'll then review it as quickly as possible. If you're interested in contributing a feature, please contact us first here before creating a pull request. We'll then decide whether we'd accept it or not. Thanks for your understanding.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant