René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"basicConfig is called at the module level","articleBody":"## Expected Behavior \r\n\r\n```\r\nimport feast\r\nlogging.basicConfig(level=level, format=FORMAT)\r\nlogging.error(\"msg\")\r\n```\r\n\r\nshould print logging message according to `FORMAT`\r\n\r\n## Current Behavior\r\n\r\nIt uses the format defined in `feast` at the module level.\r\n\r\n## Steps to reproduce\r\n\r\nSame as in \"Expected Behavior\"\r\n\r\n### Specifications\r\n\r\n- Version: 0.18.1\r\n- Platform: Linux\r\n- Subsystem: -\r\n\r\n## Possible Solution\r\n\r\nI see that `basicConfig` is called here: https://github.com/feast-dev/feast/blob/c9eda79c7b1169ef05a481a96f07960c014e88b9/sdk/python/feast/cli.py#L84 so it is possible that simply removing this call here is enough: https://github.com/feast-dev/feast/blob/0ca62970dd6bc33c00bd5d8b828752814d480588/sdk/python/feast/__init__.py#L30\r\n\r\nIf there are any other entry points that need to set up logging, they should call the function, but the call in `__init__.py` must be removed.","author":{"url":"https://github.com/elshize","@type":"Person","name":"elshize"},"datePublished":"2022-05-11T12:33:48.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":3},"url":"https://github.com/2664/feast/issues/2664"}