python-bigquery-dataframes icon indicating copy to clipboard operation
python-bigquery-dataframes copied to clipboard
verified publisher icon googleapis

[WIP] Log __init__ calls of timedelta-related ops

Open sycai opened this issue 9 months ago • 2 comments

sycai avatar Mar 28 '25 17:03 sycai

Can we please not log internal op initializers, or really anything not directly tied to user intentions or server requests? I worry this is very fragile.

I need to gather metrics on the usage of timedelta ops, and it seems this is the most straightforward approach. Could you elaborate on the fragility?

sycai avatar Mar 31 '25 22:03 sycai

Talked with @TrevorBergeron: the concern at the moment is that this PR is trying to log internal mechanisms instead of user intentions, which deviates from the purpose of the class_logger.

Putting this PR on hold for now.

sycai avatar Mar 31 '25 23:03 sycai