Skip to content
This repository has been archived by the owner on Oct 1, 2022. It is now read-only.

Replace print statements with logging #81

Open
AndrewEckart opened this issue Mar 25, 2021 · 1 comment
Open

Replace print statements with logging #81

AndrewEckart opened this issue Mar 25, 2021 · 1 comment
Assignees
@sthapa
Labels
observability 👀 Logs and metrics

Comments

@AndrewEckart
Copy link
Contributor

We should use the logging module instead of print - it allows for greater control and flexibility.
@sthapa
Copy link
Contributor

sthapa commented Mar 31, 2021 •
edited
Loading

Initial draft for transformers is at ssl-hep/ServiceX_transformer#47 . There's a a few things that we should probably discuss before finalizing the PR.

@AndrewEckart AndrewEckart added the observability 👀 Logs and metrics label May 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@sthapa sthapa

Labels
observability 👀 Logs and metrics
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sthapa @AndrewEckart