Channels
announcements
ask-marvin
best-practices-coordination-plane
data-ecosystem
data-tricks-and-tips
events
find-a-prefect-job
geo-bay-area
geo-berlin
geo-boston
geo-chicago
geo-colorado
geo-dc
geo-israel
geo-japan
geo-london
geo-nyc
geo-seattle
geo-texas
gratitude
introductions
marvin-in-the-wild
pacc-clearcover-june-12-2023
pacc-may-31-2023
ppcc-may-16-2023
prefect-ai
prefect-aws
prefect-azure
prefect-cloud
prefect-community
prefect-contributors
prefect-dbt
prefect-docker
prefect-gcp
prefect-getting-started
prefect-integrations
prefect-kubernetes
prefect-recipes
prefect-server
prefect-ui
random
show-us-what-you-got
Powered by
Title
j
Jessica Smith
05/26/2022, 1:55 PM
can we get the logger name added to FlowRunLog? Not sure why it isn't there by default, it's kind of important
k
Kevin Kho
05/26/2022, 2:22 PM
I’m not following, sorry. Could you elaborate more?
j
Jessica Smith
05/26/2022, 2:23 PM
The class FlowRunLog doesn't have a field for the logger name
https://github.com/PrefectHQ/prefect/blob/ad734292bc4113a52777d6ebe87cfa1c13a3eee8/src/prefect/backend/flow_run.py#L293
And the query doesn't request the logger name
https://github.com/PrefectHQ/prefect/blob/ad734292bc4113a52777d6ebe87cfa1c13a3eee8/src/prefect/backend/flow_run.py#L473
k
Kevin Kho
05/26/2022, 2:38 PM
This I will open a feature request for
@Marvin
open “Add logger name to FlowRunLog”
Feel free to chime in the issue
m
Marvin
05/26/2022, 2:39 PM
https://github.com/PrefectHQ/prefect/issues/5839
f
FuETL
06/27/2022, 12:24 PM
@Kevin Kho
What should be the default name? I'm thinking about working in this.
k
Kevin Kho
06/27/2022, 3:12 PM
I wouldn’t know. Worth a comment in the github issue
4 Views
#prefect-community
Join Slack