Chenyu Li
Chenyu Li
We raised better error message to tell folks not do this, but still want to leave this open to revisit and see if other better options we would like to...
@arieldbt looks great! I probably will have more time to play with it in October!
In python model we currently added [a decorator ](https://github.com/dbt-labs/dbt-core/blob/1d1f915438ac69fb1c7f13d1027ac5725046cbec/core/dbt/adapters/base/impl.py#L1279)to log things.
TODO: add a test Update: Done
@dbeatty10 This unittest [here](https://github.com/dbt-labs/dbt-core/blob/fc1fc2d5e9a0dd695846f471a8b9179d29c162aa/test/unit/test_config.py#L391) will write a profile.yml to local and test things, I think you can maybe follow things here to add an unittest for your change. Happy to...
Agreed with Emily's comment, just some clean up then this is good to go!
@jtcohen6 either way feels not very easy, any idea?
Talked about this live, the manipulating `context` idea might be the better approach
@jtcohen6 @lostmygithubaccount I think I found a path to modify the context so that we only allow the code(not saying I like it) https://github.com/dbt-labs/dbt-core/blob/a1ee348a6f9af3ffc88ea8100c782adc94091f44/core/dbt/clients/jinja.py#L301-L321 The `trace_call` function here is called...
@cla-bot check